Use new earcons in ChromeVox Next.

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

Index: chrome/browser/resources/chromeos/chromevox/cvox2/background/output_test.extjs
diff --git a/chrome/browser/resources/chromeos/chromevox/cvox2/background/output_test.extjs b/chrome/browser/resources/chromeos/chromevox/cvox2/background/output_test.extjs
index ad642ed0e50f3fac32d7db146e9440a999f107be..de96eaba32459511a2b0a32eeed8df7b9b1e59a5 100644
--- a/chrome/browser/resources/chromeos/chromevox/cvox2/background/output_test.extjs
+++ b/chrome/browser/resources/chromeos/chromevox/cvox2/background/output_test.extjs
@@ -108,14 +108,15 @@ TEST_F('OutputE2ETest', 'Checkbox', function() {
       var el = root.firstChild.firstChild;
       var range = cursors.Range.fromNode(el);
       var o = new Output().withSpeechAndBraille(range, null, 'navigate');
-      assertEqualsJSON({string_: '|Check box|not checked', 'spans_': [
-        // Attributes.
-        {value: 'name', start: 0, end: 0},
-        {value: 'role', start: 1, end: 10},
-        {value: 'state', start: 11, end: 22},
-
+      assertEqualsJSON({string_: '||Check box|not checked', 'spans_': [
         // Checkbox earcon (based on the state).
-        {value: {earconId: 'CHECK_OFF'}, start: 11, end: 22}
+        {value: {earconId: 'CHECK_OFF'}, start: 0, end: 0},
+
+        // Attributes.
+        {value: 0, start: 0, end: 0},

[Peter Lundblad, 2015/09/29 09:27:34]

Where does this span of the number 0 come from and what does it mean?

[dmazzoni, 2015/10/01 20:41:43]

Oops, I thought that was because I got rid of the "if (buff.toString())" check
at the top of go() - but I also had a typo in the code to append the earcons.
Fixed.

+        {value: 'name', start: 1, end: 1},
+        {value: 'role', start: 2, end: 11},
+        {value: 'state', start: 12, end: 23}
       ]}, o.speechOutputForTest);
       checkBrailleOutput(
           'chk ( )',
@@ -223,9 +224,11 @@ TEST_F('OutputE2ETest', 'Audio', function() {
       var prevRange = range;
       range = cursors.Range.fromNode(el);
       var o = new Output().withSpeechAndBraille(range, prevRange, 'navigate');
-      assertEqualsJSON({string_: '0, , slider|audio time scrubber',
+      assertEqualsJSON({string_: '|0, , slider|audio time scrubber',
           spans_:
-              [{value: 'help', start: 12, end: 31}]
+              [{value: {'earconId': 'SLIDER'}, start: 0, end: 0},
+               {value: 0, start: 0, end: 0},

[Peter Lundblad, 2015/09/29 09:27:34]

Where does 0 come from?

[dmazzoni, 2015/10/01 20:41:43]

Same bug as above. Fixed.

+               {value: 'help', start: 13, end: 32}]
           }, o.speechOutputForTest);
       // TODO(plundblad): Investigate this.
       checkBrailleOutput(
@@ -453,6 +456,7 @@ TEST_F('OutputE2ETest', 'Menu', function() {
     assertEqualsJSON({string_:
             '|Menu|with 1 item|a|Menu item| 1 of 1 ', spans_: [
         {value: 'name', start: 18, end: 19},
+        {value: {earconId: 'BUTTON'}, start:18, end:19},
         {value: 'role', start:20, end: 29}
     ]}, o.speechOutputForTest);
     checkBrailleOutput(