Complete live region support in ChromeVox Next.

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

Index: chrome/browser/resources/chromeos/chromevox/cvox2/background/live_regions.js
diff --git a/chrome/browser/resources/chromeos/chromevox/cvox2/background/live_regions.js b/chrome/browser/resources/chromeos/chromevox/cvox2/background/live_regions.js
new file mode 100644
index 0000000000000000000000000000000000000000..8daf2fa57f4bc4736f13d0f4f530ca238fc8c59e
--- /dev/null
+++ b/chrome/browser/resources/chromeos/chromevox/cvox2/background/live_regions.js
@@ -0,0 +1,146 @@
+// 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 Implements support for live regions in ChromeVox Next.
+ */
+
+goog.provide('LiveRegions');
+
+goog.require('ChromeVoxState');
+
+/**
+ * ChromeVox2 live region handler.
+ * @param {ChromeVoxState} chromeVoxState The ChromeVox state object,
+ *     keeping track of the current mode and current range.
+ * @constructor
+ */
+LiveRegions = function(chromeVoxState) {
+  /**
+   * @type {ChromeVoxState}
+   * @private
+   */
+  this.chromeVoxState_ = chromeVoxState;
+
+  /**
+   * The time the last live region event was output.
+   * @type {Date}
+   * @private
+   */
+  this.lastLiveRegionTime_ = new Date(0);
+
+  /**
+   * Set of nodes that have been announced as part of a live region since
+   *     |this.lastLiveRegionTime_|, to prevent duplicate announcements.
+   * @type {WeakSet<chrome.automation.AutomationNode>}
+   * @private
+   */
+  this.liveRegionNodeSet_ = new WeakSet();
+
+  chrome.automation.setTreeChangeObserverMask('liveRegionTreeChanges');
+  chrome.automation.addTreeChangeObserver(this.onTreeChange.bind(this));
+};
+
+/**
+ * Live region events received in fewer than this many milliseconds will
+ * queue, otherwise they'll be output with a category flush.
+ * @type {number}
+ * @const
+ */
+LiveRegions.LIVE_REGION_QUEUE_TIME_MS = 500;
+
+LiveRegions.prototype = {
+  /**
+   * Called when the automation tree is changed.
+   * @param {chrome.automation.TreeChange} treeChange

[Peter Lundblad, 2015/11/24 11:04:31]

Suggest using goog.scope like in some other files to make type names less
verbose.

[dmazzoni, 2015/11/30 22:00:47]

Done.

+   */
+  onTreeChange: function(treeChange) {
+    var node = treeChange.target;
+    if (!node.containerLiveStatus)
+      return;
+
+    var mode = this.chromeVoxState_.getMode();
+    var currentRange = this.chromeVoxState_.getCurrentRange();
+
+    if (mode === ChromeVoxMode.CLASSIC || !cvox.ChromeVox.isActive)
+      return;
+
+    if (!currentRange)
+      return;
+
+    if (!AutomationUtil.isInSameWebpage(node, currentRange.start.node))

[Peter Lundblad, 2015/11/24 11:04:31]

Doesn't classic announce live region updates from background tabs?
I am not sure I personally think it is the right behaviour, but I think it has
been requested in the past and changed according to such request.

[dmazzoni, 2015/11/30 22:00:47]

Good catch.

I switched it but made it an option so it's easy to flip and maybe make it a
preference in the future.

+      return;
+
+    var type = treeChange.type;
+    var relevant = node.containerLiveRelevant;
+    if (relevant.indexOf('additions') >= 0 &&
+        (type == 'nodeCreated' || type == 'subtreeCreated')) {
+      this.outputLiveRegionChange_(node, null);
+    }
+
+    if (relevant.indexOf('text') >= 0 && type == 'nodeChanged')
+      this.outputLiveRegionChange_(node, null);
+
+    if (relevant.indexOf('removals') >= 0 && type == 'nodeRemoved')
+      this.outputLiveRegionChange_(node, '@live_regions_removed');
+  },
+
+  /**
+   * Given a node that needs to be spoken as part of a live region
+   * change and an additional optional format string, output the
+   * live region description.
+   * @param {!chrome.automation.AutomationNode} node The changed node.
+   * @param {?string} opt_prependFormatStr If set, a format string for

[Peter Lundblad, 2015/11/24 11:04:31]

Optional parameters should have an = suffix.

[dmazzoni, 2015/11/30 22:00:47]

Done.

+   *     cvox2.Output to prepend to the output.
+   * @private
+   */
+  outputLiveRegionChange_: function(node, opt_prependFormatStr) {
+    if (node.containerLiveBusy)
+      return;
+
+    if (node.containerLiveAtomic && !node.liveAtomic) {
+      if (node.parent)
+        this.outputLiveRegionChange_(node.parent, opt_prependFormatStr);
+      return;
+    }
+
+    var range = cursors.Range.fromNode(node);
+    var output = new Output();
+    if (opt_prependFormatStr) {

[Peter Lundblad, 2015/11/24 11:04:31]

nit: bracket inconsistency.

[dmazzoni, 2015/11/30 22:00:47]

Done.

+      output.format(opt_prependFormatStr);
+    }
+    output.withSpeech(range, range, Output.EventType.NAVIGATE);
+
+    if (!output.hasSpeech && node.liveAtomic) {

[Peter Lundblad, 2015/11/24 11:04:31]

Ditto.

[dmazzoni, 2015/11/30 22:00:47]

Done.

+      output.format('$descendants', node);
+    }
+
+    output.withSpeechCategory(cvox.TtsCategory.LIVE);
+
+    if (!output.hasSpeech)
+      return;
+
+    // Enqueue live region updates that were received at approximately
+    // the same time, otherwise flush previous live region updates.
+    var currentTime = new Date();
+    var queueTime = LiveRegions.LIVE_REGION_QUEUE_TIME_MS;
+    if (currentTime - this.lastLiveRegionTime_ > queueTime) {
+      this.liveRegionNodeSet_ = new WeakSet();
+      output.withQueueMode(cvox.QueueMode.CATEGORY_FLUSH);
+      this.lastLiveRegionTime_ = currentTime;
+    } else {
+      output.withQueueMode(cvox.QueueMode.QUEUE);
+    }
+
+    var parent = node;
+    while (parent) {
+      if (this.liveRegionNodeSet_.has(parent))
+        return;
+      parent = parent.parent;
+    }
+
+    this.liveRegionNodeSet_.add(node);
+    output.go();
+  },
+};