Revert of Reland #3: Ensure WebView notifies desktop automation on creation, (patchs...
commit0672f1e23fe1825ecc849a6ac66bab4ffbf43fbf
authordtseng <dtseng@chromium.org>
Wed, 11 Feb 2015 18:07:39 +0000 (11 10:07 -0800)
committerCommit bot <commit-bot@chromium.org>
Wed, 11 Feb 2015 18:08:11 +0000 (11 18:08 +0000)
treeafedab1e454d4ff0454042c521cd355d1afbe9df
parentfb8763f2c07cb72079b63a183f0afc9d863937b9
Revert of Reland #3: Ensure WebView notifies desktop automation on creation,  (patchset #2 id:20001 of https://codereview.chromium.org/907973004/)

Reason for revert:
Still flakes...

Original issue's description:
> Reland #3: Ensure WebView notifies desktop automation on creation, destruction, and change
>
> Original issue: https://codereview.chromium.org/880063002
> Previous reland attempt: https://codereview.chromium.org/895623003/
> previous reland attempt 2: https://codereview.chromium.org/890013006/
> - able to reproduce the flakes locally by running multiple copies of the same test in parallel.
> - tracked the issues down to the following points/solutions:
> 1. in automation_node.js, we now fire a loadComplete event if we detect a loaded root within enableFrame's callback. Previously, there could be a situation where a tree was already finished loading so it never actually fires that load complete.
> 2. in automation_node.js, we now store the callback that fires childrenChanged and the loadComplete above before requesting the browser initiate tree only accessibility which sends down the data.
> 3. in chromevox_next_e2e_test_base.js, we now check that the url contains 'test' to avoid processing loadCompletes from other rootWebAreas (e.g. about:blank).
>
> TBR=dmazzoni@chromium.org
>
> Committed: https://crrev.com/4da650e88c859e6d1c2d82993cd6ed3376e77f80
> Cr-Commit-Position: refs/heads/master@{#315785}

TBR=dmazzoni@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true

Review URL: https://codereview.chromium.org/891633004

Cr-Commit-Position: refs/heads/master@{#315794}
20 files changed:
chrome/browser/resources/chromeos/chromevox/common/chrome_extension_externs.js
chrome/browser/resources/chromeos/chromevox/common/editable_text_base.js
chrome/browser/resources/chromeos/chromevox/cvox2/background/automation_util.js
chrome/browser/resources/chromeos/chromevox/cvox2/background/automation_util_test.extjs
chrome/browser/resources/chromeos/chromevox/cvox2/background/background.js
chrome/browser/resources/chromeos/chromevox/cvox2/background/background_test.extjs
chrome/browser/resources/chromeos/chromevox/cvox2/background/cursors_test.extjs
chrome/browser/resources/chromeos/chromevox/cvox2/background/output.js
chrome/browser/resources/chromeos/chromevox/cvox2/background/output_test.extjs
chrome/browser/resources/chromeos/chromevox/testing/chromevox_e2e_test_base.js
chrome/browser/resources/chromeos/chromevox/testing/chromevox_next_e2e_test_base.js
chrome/browser/resources/chromeos/chromevox/testing/mock_tts.js
chrome/renderer/resources/extensions/automation/automation_node.js
chrome/test/base/extension_js_browser_test.h
chrome/test/base/extension_load_waiter_one_shot.h
ui/views/accessibility/ax_aura_obj_cache.cc
ui/views/accessibility/ax_aura_obj_cache.h
ui/views/accessibility/ax_widget_obj_wrapper.cc
ui/views/controls/webview/webview.cc
ui/views/controls/webview/webview.h