Refine braille output

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 bcb151d1fb64a6cf858eccda24c9e6b1f5cf0322..b5e7e04b5caa722173b4406f1c49c903156c3b8b 100644
--- a/chrome/browser/resources/chromeos/chromevox/cvox2/background/output_test.extjs
+++ b/chrome/browser/resources/chromeos/chromevox/cvox2/background/output_test.extjs
@@ -278,7 +278,7 @@ TEST_F('OutputE2ETest', 'Input', function() {
         ['||Spin button',
          [{value: 'name', start: 0, end: 0},
           {value: new Output.EarconAction('LISTBOX'), start: 0, end: 0},
-          {value: {startIndex: 0, endIndex: 0, offset: 0}, start: 1, end: 1},
+          {value: {startIndex: 0, endIndex: 0}, start: 1, end: 1},
           {value: 'value', start: 1, end: 1},
           {value: 'role', start: 2, end: 13}]
         ],
@@ -300,16 +300,16 @@ TEST_F('OutputE2ETest', 'Input', function() {
       // TODO(plundblad): Some of these are wrong, there should be an initial
       // space for the cursor in edit fields.
       var expectedBrailleValues = [
-        ' ed',
-        ' @ed',
-        ' pwded',
-        ' #ed',
-        ' spnbtn',
+        '  ed',
+        '  @ed',
+        '  pwded',
+        '  #ed',
+        '  spnbtn',
         {string_: 'time'},
         {string_: 'date'},
         {string_: 'Choose File No file chosen btn'},
-        ' srched',
-        ' ed'
+        '  srched',
+        '  ed'
       ];
       assertEquals(expectedSpeechValues.length, expectedBrailleValues.length);
 
@@ -333,7 +333,7 @@ TEST_F('OutputE2ETest', 'Input', function() {
         if (typeof expectedValue === 'string') {
           checkBrailleOutput(
               expectedValue,
-              [{value: {startIndex: 0, endIndex: 0, offset: 0},
+              [{value: {startIndex: 0, endIndex: 0},
                 start: 0, end: 0},
                {value: new Output.NodeSpan(el), start: 0,
                 end: expectedValue.length}],
@@ -668,3 +668,26 @@ TEST_F('OutputE2ETest', 'ContainerFocus', function() {
           new Output().withSpeech(r1, r2).speechOutputForTest.string_);
   });
 });
+
+TEST_F('OutputE2ETest', 'BraileWhitespace', function() {

[dmazzoni, 2016/11/10 23:31:29]

nit: Braile -> Braille

+  this.runWithLoadedTree(function() {/*!
+    <p>this is a <em>test</em>of emphasized text</p>
+  */},
+  function(root) {
+    var start = root.firstChild.firstChild;
+    var end = root.firstChild.lastChild;
+    var range = new cursors.Range(
+        cursors.Cursor.fromNode(start),
+        cursors.Cursor.fromNode(end)
+        );
+    var o = new Output().withBraille(range, null, 'navigate');
+    checkBrailleOutput(
+        'this is a testof emphasized text',
+        [
+          {value: new Output.NodeSpan(start), start: 0, end: 10},
+          {value: new Output.NodeSpan(start.nextSibling), start: 10, end: 14},
+          {value: new Output.NodeSpan(end), start: 14, end: 32}
+        ],
+        o);
+  });
+});