Implement ChromeVox next/previous line, link, and heading.

chrome/browser/resources/chromeos/chromevox/cvox2/background/background.extjs

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // Include test fixture.
 GEN_INCLUDE(['../../testing/chromevox_e2e_test_base.js']);
 
 /**
  * Test fixture for cvox2.Background.
  * @constructor
@@ skipping 31 matching lines @@
 
 /**
  * Mock tts class.
  * @constructor
  * @extends {cvox.TtsInterface}
  */
 var MockTts = function() {
 };
 
 MockTts.prototype = {
-  /** Tracks all spoken text. @type {!Array.<string>} */
-  utterances: [],
+  /**
+   * A list of predicate callbacks.
+   * @type {!Array.<function(string) : boolean>}
+   * @private
+   */
+  callbacks_: [],
+
+  /**
+   * A list of strings stored whenever there are no expectations.
+   * @type {!Array.<string}
+   * @private
+   */
+idleUtterances_: [],

[dmazzoni, 2014/09/29 06:06:17]

nit: indent 2

[David Tseng, 2014/09/29 17:05:23]

Done.

 
   /** @override */
   speak: function(textString, queueMode, properties) {
-    this.utterances.push(textString);
+    this.process_(textString);
   },
 
   /**
-   * Checks to see if a string was spoken.
-   * @param {string} textString The string to check.
-   * @return {boolean} True if the string was spoken (possibly as part of a
-   * larger utterance).
+   * Adds an expectation for the given string to be spoken. If satisfied,
+   * |opt_callback| is * called.

[dmazzoni, 2014/09/29 06:06:18]

Nit: delete extra '*'

[David Tseng, 2014/09/29 17:05:23]

Done.

+   * @param {string} expected
+   * @param {function() : void=} opt_callback
    */
-  checkIfSubstringWasSpoken: function(textString) {
-    return this.utterances.some(function(t) {
-      return t.indexOf(textString) != -1;
+  expectSpeech: function(expected, opt_callback) {
+    this.callbacks_.push(function(actual) {
+      var match = actual.indexOf(expected) != -1;
+      if (opt_callback && match)
+        opt_callback();
+      return match;
     });
-  }
+
+    // Process any idleUtterances.
+    this.idleUtterances_.forEach(this.process_, true);
+  },
+
+  /**
+   * @param {string} textString Utterance to match against callbacks.
+   * @param {boolean=} opt_manual True if called outside of tts.speak.
+   * @private
+   */
+  process_: function(textString, opt_manual) {
+    if (this.callbacks_.length == 0) {
+      if (!opt_manual)
+        this.idleUtterances_.push(textString);
+      return;
+    }
+
+    var allUtterances = this.idleUtterances_.concat([textString]);
+    var targetCallback = this.callbacks_.shift();
+    if (allUtterances.some(targetCallback))
+      this.idleUtterances_.length = 0;
+    else
+      this.callbacks_.unshift(targetCallback);
+  },
 };
 
 /** Tests that ChromeVox classic is in this context. */
 SYNC_TEST_F('BackgroundTest', 'ClassicNamespaces', function() {
   assertEquals('object', typeof(cvox));
   assertEquals('function', typeof(cvox.ChromeVoxBackground));
 });
 
 /** Tests that ChromeVox next is in this context. */
 SYNC_TEST_F('BackgroundTest', 'NextNamespaces', function() {
@@ skipping 12 matching lines @@
     for (var i = 0; i < root.children().length; i++) {
       var found = findStatusTray(root.children()[i]);
       if (found)
         return found;
     }
     return null;
   }
 
   chrome.automation.getDesktop(function(root) {
     var testButton = findStatusTray(root);
-    testButton.addEventListener(chrome.automation.EventType.focus,
-        function(e) {
-          var result =
-              cvox.ChromeVox.tts.checkIfSubstringWasSpoken('Status tray');
-          testDone([result, '']);
-        },
-        true);
+    cvox.ChromeVox.tts.expectSpeech('Status tray', testDone);

[dmazzoni, 2014/09/29 06:06:18]

This looks much cleaner. I don't mind that the order of these is reversed.

     testButton.focus();
   });
 });
+
+/** Tests feedback once a page loads. */
+TEST_F('BackgroundTest', 'InitialFeedback', function() {
+  this.runWithDocument(function() {/*!
+    <p>start
+    <p>end
+  */},
+  function() {
+    cvox.ChromeVox.tts.expectSpeech('start', function() {
+      cvox2.global.backgroundObj.onGotCommand('nextLine');
+    });
+    cvox.ChromeVox.tts.expectSpeech('end', testDone);
+  }.bind(this));
+});