Finish implementing ChromeVox Next active indicator.

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

 // Copyright 2015 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.
 
 /**
  * @fileoverview Handles automation from a desktop automation node.
  */
 
 goog.provide('DesktopAutomationHandler');
 
@@ skipping 232 matching lines @@
     // Value change events fire on web text fields and text areas when pressing
     // enter; suppress them.
     if (!global.backgroundObj.currentRange ||
         evt.target.role != RoleType.textField) {
       this.onEventDefault(evt);
       global.backgroundObj.currentRange = cursors.Range.fromNode(evt.target);
     }
   },
 
   /**
+   * 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
    */
   createEditableTextHandlerIfNeeded_: function(node) {
     if (!this.editableTextHandler_ ||
         node != global.backgroundObj.currentRange.start.node) {
       var start = node.textSelStart;
       var end = node.textSelEnd;
       if (start > end) {
         var tempOffset = end;
@@ skipping 20 matching lines @@
   if (cvox.ChromeVox.isMac)
     return;
   chrome.automation.getDesktop(function(desktop) {
     global.desktopAutomationHandler = new DesktopAutomationHandler(desktop);
   });
 };
 
 DesktopAutomationHandler.init_();
 
 });  // goog.scope