Reuse category flush when outputing live regions.

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

 // 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.
 
 /**
  * @fileoverview Provides output services for ChromeVox.
  */
 
 goog.provide('Output');
 goog.provide('Output.EventType');
@@ skipping 627 matching lines @@
    * @param {chrome.automation.EventType|Output.EventType} type
    * @return {!Output}
    */
   withSpeechAndBraille: function(range, prevRange, type) {
     this.withSpeech(range, prevRange, type);
     this.withBraille(range, prevRange, type);
     return this;
   },
 
   /**
+   * Applies the given speech category to the output.
+   * @param {cvox.TtsCategory} category
+   * @return {!Output}
+   */
+  withSpeechCategory: function(category) {
+    this.speechProperties_['category'] = category;
+    return this;
+  },
+
+  /**
    * Apply a format string directly to the output buffer. This lets you
    * output a message directly to the buffer using the format syntax.
    * @param {string} formatStr
    * @return {!Output}
    */
   format: function(formatStr) {
     this.formatOptions_ = {speech: true, braille: false, location: true};
     this.format_(null, formatStr, this.speechBuffer_);
 
     this.formatOptions_ = {speech: false, braille: true, location: false};
@@ skipping 12 matching lines @@
         callback();
     }.bind(this);
     return this;
   },
 
   /**
    * Executes all specified output.
    */
   go: function() {
     // Speech.
-    var queueMode = cvox.QueueMode.FLUSH;
+    var queueMode = cvox.QueueMode.CATEGORY_FLUSH;

[dmazzoni, 2015/10/13 20:20:40]

Should this always be category flush? How about only if there's a category?

[David Tseng, 2015/10/13 21:27:59]

Done.

     this.speechBuffer_.forEach(function(buff, i, a) {
       if (buff.toString()) {
         (function() {
           var scopedBuff = buff;
           this.speechProperties_['startCallback'] = function() {
             var actions = scopedBuff.getSpansInstanceOf(Output.Action);
             if (actions) {
               actions.forEach(function(a) {
                 a.run();
               });
@@ skipping 629 matching lines @@
             elem.end);
       });
       spansToRemove.forEach(result.removeSpan.bind(result));
       separator = Output.SPACE;
     });
     return result;
   }
 };
 
 });  // goog.scope