Revert of Reland #2: Ensure WebView notifies desktop automation

Revert of Reland #2: Ensure WebView notifies desktop automation on creation, destruction, and change  Original (patchset #7 id:120001 of https://codereview.chromium.org/890013006/)

Reason for revert:
Suspected to cause chromeos vox test failure:
http://build.chromium.org/p/chromium.chromiumos/builders/Linux%20ChromiumOS%20Tests%20%281%29/builds/56046

Original issue's description:
> Reland #2: 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/
>
> This cl changes the way in which ChromeVox initially verbalizes a page which led to flakeyness in
>   chromevox_tests --gtest_filter=BackgroundTest.*
>
> To attempt and address the flakeyness,
> - chromevox_e2e_test_base.js and chromevox_next_e2e_test_base.js now explicitly separates listening to page load events from the chrome.automation tree and the chrome.tabs APIs.
> They are named: runWithLoadedTree, runWithLoadedTab, and runWithTab. The last method allows a caller to create a tab, but not wait for it to load (based on the tab status being 'complete').
> - each test in background_test now no longer waits for ChromeVox to speak 'start' (which formerly could have been triggered by a loadComplete automation event).
> - tab creation is still done via the tabs API, but listening for load is done via chrome.automation. (i.e. add a listener for an automation tree to load completely, then create a tab).
>
> It is not known at this time if this cl will continue to cause flakes in the above tests. Try bots and local runs yielded no flakeyness. The next step may be to introduce logging to identify the source of the flakeyness.
>
> TBR=tfarina
>
> Committed: https://crrev.com/9b006b56ca402da3774708f08e062c89018d57a6
> Cr-Commit-Position: refs/heads/master@{#315362}

TBR=dmazzoni@chromium.org,tfarina@chromium.org,dtseng@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true

Committed: https://crrev.com/4e8f4c06c982650d51fef832506002cebe78d8fb
Cr-Commit-Position: refs/heads/master@{#315373}

[jiayl, 2015-02-09 20:16:54]

Created Revert of Reland #2: Ensure WebView notifies desktop automation on
creation, destruction, and change  Original

[jiayl, 2015-02-09 20:51:01]

jiayl@chromium.org changed reviewers:
+ sky@chromium.org

[jiayl, 2015-02-09 20:51:40]

The CQ bit was checked by jiayl@chromium.org

[commit-bot: I haz the power, 2015-02-09 20:52:48]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/910013002/1

[commit-bot: I haz the power, 2015-02-09 20:54:30]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-02-09 20:54:31]

Failed to apply patch for
chrome/browser/resources/chromeos/chromevox/cvox2/background/background_test.extjs:
While running git apply --index -3 -p1;
  error: patch failed:
chrome/browser/resources/chromeos/chromevox/cvox2/background/background_test.extjs:92
  Falling back to three-way merge...
  Applied patch to
'chrome/browser/resources/chromeos/chromevox/cvox2/background/background_test.extjs'
with conflicts.
  U
chrome/browser/resources/chromeos/chromevox/cvox2/background/background_test.extjs

Patch:      
chrome/browser/resources/chromeos/chromevox/cvox2/background/background_test.extjs
Index:
chrome/browser/resources/chromeos/chromevox/cvox2/background/background_test.extjs
diff --git
a/chrome/browser/resources/chromeos/chromevox/cvox2/background/background_test.extjs
b/chrome/browser/resources/chromeos/chromevox/cvox2/background/background_test.extjs
index
08333701e8ac700ed0a0c316c51e7bf7076b0548..7ed70f7cf4bc690fd7ee01e4e34a258d858f5d19
100644
---
a/chrome/browser/resources/chromeos/chromevox/cvox2/background/background_test.extjs
+++
b/chrome/browser/resources/chromeos/chromevox/cvox2/background/background_test.extjs
@@ -92,17 +92,22 @@
 
 /** Tests feedback once a page loads. */
 TEST_F('BackgroundTest', 'InitialFeedback', function() {
-  cvox.ChromeVox.tts.expectSpeech('start', testDone);
-
-  this.runWithTab(function() {/*!
+  cvox.ChromeVox.tts.expectSpeech('start', function() {
+    global.backgroundObj.onGotCommand('nextLine');
+  }, true);
+  cvox.ChromeVox.tts.expectSpeech('end', testDone, true);
+
+  this.runWithDocument(function() {/*!
     <p>start
     <p>end
-  */});
+  */},
+  function() {});
 });
 
 /** Tests consistency of navigating forward and backward. */
 TEST_F('BackgroundTest', 'ForwardBackwardNavigation', function() {
-  this.runWithLoadedTree(this.linksAndHeadingsDoc, function() {
+  cvox.ChromeVox.tts.expectSpeech('start', null, true);
+  this.runWithDocument(this.linksAndHeadingsDoc, function() {
       var doCmd = this.doCmd.bind(this);
       var expectAfter =
           cvox.ChromeVox.tts.expectSpeechAfter.bind(cvox.ChromeVox.tts);
@@ -132,7 +137,8 @@
 });
 
 TEST_F('BackgroundTest', 'CaretNavigation', function() {
-  this.runWithLoadedTree(this.linksAndHeadingsDoc, function() {
+  cvox.ChromeVox.tts.expectSpeech('start', null, true);
+  this.runWithDocument(this.linksAndHeadingsDoc, function() {
       var doCmd = this.doCmd.bind(this);
       var expectAfter =
           cvox.ChromeVox.tts.expectSpeechAfter.bind(cvox.ChromeVox.tts);
@@ -162,7 +168,7 @@
 
 // Flaky: http://crbug.com/451362
 TEST_F('BackgroundTest', 'DISABLED_SelectSingleBasic', function() {
-  this.runWithLoadedTree(this.formsDoc, function(tabId) {
+  this.runWithDocument(this.formsDoc, function(tabId) {
     var sendDownToSelect =
         this.sendKeyToElement.bind(this, tabId, 'Down', '#fruitSelect');
     var expect = cvox.ChromeVox.tts.expectSpeech.bind(cvox.ChromeVox.tts);
@@ -174,7 +180,8 @@
 });
 
 TEST_F('BackgroundTest', 'ContinuousRead', function() {
-  this.runWithLoadedTree(this.linksAndHeadingsDoc, function() {
+  cvox.ChromeVox.tts.expectSpeech('start', null, true);
+  this.runWithDocument(this.linksAndHeadingsDoc, function() {
     var doCmd = this.doCmd.bind(this);
     var expect =
         cvox.ChromeVox.tts.expectSpeechAfter.bind(cvox.ChromeVox.tts);

[commit-bot: I haz the power, 2015-02-09 20:54:47]

The CQ bit was unchecked by commit-bot@chromium.org