Finish implementing ChromeVox Next active indicator.

chrome/browser/resources/chromeos/chromevox/cvox2/background/desktop_automation_handler.js

Index: chrome/browser/resources/chromeos/chromevox/cvox2/background/desktop_automation_handler.js
diff --git a/chrome/browser/resources/chromeos/chromevox/cvox2/background/desktop_automation_handler.js b/chrome/browser/resources/chromeos/chromevox/cvox2/background/desktop_automation_handler.js
index 49c4416d8d3725eca09a7b4c40db6b6770bb1b8c..847ccdf8efdfe45fd50ae381953cdd7859b2b70b 100644
--- a/chrome/browser/resources/chromeos/chromevox/cvox2/background/desktop_automation_handler.js
+++ b/chrome/browser/resources/chromeos/chromevox/cvox2/background/desktop_automation_handler.js
@@ -250,6 +250,16 @@ DesktopAutomationHandler.prototype = {
   },
 
   /**
+   * Handle updating the active indicator when the document scrolls.
+   * @param {Object} evt
+   */

[Peter Lundblad, 2015/11/20 11:15:03]

@override

[dmazzoni, 2015/11/20 23:35:14]

Done.

+  onScrollPositionChanged: function(evt) {
+    new Output().withLocation(
+            global.backgroundObj.currentRange, null, evt.type)

[Peter Lundblad, 2015/11/20 11:15:03]

currentRange can be null, but this parameter is not nullable. Not sure why the
compiler doesn't catch that.

[dmazzoni, 2015/11/20 23:35:14]

Huh. Added a null check.

+        .go();
+  },
+
+  /**
    * Create an editable text handler for the given node if needed.
    * @param {Object} node
    */