Fix issues with sending tts queue lots of utterances

- $descendants for div's can be very expensive. For example, in Gmail, there is a focusable div that wraps almost the entire contents of the page. We end up generating Output for everything under the div and lag significantly when focus lands here.
Fix this by only taking the text content of divs as one large string.  $descendants had all output including rich formatting.
- live region output on alerts causes double output (and similar lag). For example, pressing '?' in Gmail triggers hundreds of runs inside of LiveRegion.
Fix this by ignoring live regions set on alert nodes.

TEST=navigate in Gmail.
BUG=672955
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2563173002
Cr-Commit-Position: refs/heads/master@{#437925}

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

@@ -997,7 +997,7 @@ TEST_F('BackgroundTest', 'ContentEditableJumpSyncsRange', function() {
     };
 
     mockFeedback.call(doCmd('nextEditText'))
-        .expectSpeech('Top News')
+        .expectSpeech('Top News Most Popular Sports')
         .call(doCmd('nextHeading'))
         .expectSpeech('Top News')
         .call(assertRangeHasText('Top News'))

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

@@ -134,6 +134,10 @@ LiveRegions.prototype = {
       return;
     }
 
+    // Alerts should be announced as a result of focus.
+    if (node.role == RoleType.alert)
+      return;
+
     var range = cursors.Range.fromNode(node);
     var output = new Output();
     if (opt_prependFormatStr)

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

@@ -109,9 +109,7 @@ TEST_F('LiveRegionsTest', 'LiveRegionChangeAtomic', function() {
     function(rootNode) {
       var go = rootNode.find({ role: RoleType.button });
       mockFeedback.call(go.doDefault.bind(go))
-          .expectCategoryFlushSpeech('Alpha')
-          .expectQueuedSpeech('Bravo')
-          .expectQueuedSpeech('Charlie');
+          .expectCategoryFlushSpeech('Alpha Bravo Charlie')
       mockFeedback.replay();
     });
 });

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

@@ -397,11 +397,12 @@ Output.RULES = {
       leave: '@exited_container($role)'
     },
     alert: {
-      speak: '$earcon(ALERT_NONMODAL) $role $descendants $state'
+      enter: '$name $role $state',
+      speak: '$earcon(ALERT_NONMODAL) $role $nameOrTextContent $state'
     },
     alertDialog: {
       enter: '$earcon(ALERT_MODAL) $name $state',
-      speak: '$earcon(ALERT_MODAL) $name $descendants $state $role'
+      speak: '$earcon(ALERT_MODAL) $name $nameOrTextContent $state $role'
     },
     cell: {
       enter: '@cell_summary($tableCellRowIndex, $tableCellColumnIndex) ' +
@@ -524,7 +525,7 @@ Output.RULES = {
       enter: '$node(tableRowHeader)'
     },
     rowHeader: {
-      speak: '$descendants $state'
+      speak: '$nameOrTextContent $state'
     },
     slider: {
       speak: '$earcon(SLIDER) @describe_slider($value, $name) $description ' +
@@ -547,7 +548,7 @@ Output.RULES = {
           '$node(tableHeader)'
     },
     tableHeaderContainer: {
-      speak: '$descendants $state $description'
+      speak: '$nameOrTextContent $state $description'
     },
     textField: {
       speak: '$name $value $if($multiline, @tag_textarea, $if(' +
@@ -601,7 +602,7 @@ Output.RULES = {
   alert: {
     default: {
       speak: '$earcon(ALERT_NONMODAL) @role_alert ' +
-          '$if($name, $name, $descendants) $description'
+          '$nameOrTextContent $description'
     }
   }
 };
@@ -1236,11 +1237,14 @@ Output.prototype = {
                 Dir.FORWARD,
                 {visit: AutomationPredicate.leafOrStaticText,
                  leaf: AutomationPredicate.leafOrStaticText});
+            var outputStrings = [];
             while (walker.next().node &&
                 walker.phase == AutomationTreeWalkerPhase.DESCENDANT) {
               if (walker.node.name)
-                this.append_(buff, walker.node.name, options);
+                outputStrings.push(walker.node.name);
             }
+            var joinedOutput = outputStrings.join(' ');
+            this.append_(buff, joinedOutput, options);
           }
         } else if (node[token] !== undefined) {
           options.annotation.push(token);