Complete table support in ChromeVox Next.

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

 // 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.
 
 // Include test fixture.
 GEN_INCLUDE(['../../testing/chromevox_next_e2e_test_base.js',
              '../../testing/assert_additions.js']);
 
 GEN_INCLUDE(['../../testing/mock_feedback.js']);
 
@@ skipping 949 matching lines @@
         .call(assertRangeHasText('Top News'))
         .call(doCmd('nextHeading'))
         .expectSpeech('Most Popular')
         .call(assertRangeHasText('Most Popular'))
         .call(doCmd('nextLine'))
         .expectSpeech('Sports')
         .call(assertRangeHasText('Sports'))
         .replay();
   });
 });
+
+TEST_F('BackgroundTest', 'BasicTableCommands', function() {

[dmazzoni, 2016/07/13 23:38:28]

Could you add a test where a cell is missing? For example, say you want to move
to the last cell but the table doesn't have that cell - I don't think we're
handling that case now but I think it should return the last cell that's
actually in the table rather than only jumping to cell (M, N) if it exists and
returning an error otherwise.

[David Tseng, 2016/07/14 20:24:41]

Done.

+  var mockFeedback = this.createMockFeedback();
+  this.runWithLoadedTree(function(root) {/*!
+  <table border=1>
+    <tr><td>name</td><td>title</td><td>address</td><td>phone</td></tr>
+    <tr><td>Dan</td><td>Mr</td><td>666 Elm Street</td><td>212 222 5555</td></tr>
+  </table>
+  */}, function(root) {
+    mockFeedback.call(doCmd('nextRow'))
+        .expectSpeech('Dan', 'row 2 column 1')
+        .call(doCmd('previousRow'))
+        .expectSpeech('name', 'row 1 column 1')
+        .call(doCmd('previousRow'))
+        .expectSpeech('No cell above.')
+        .call(doCmd('nextCol'))
+        .expectSpeech('title', 'row 1 column 2')
+        .call(doCmd('nextRow'))
+        .expectSpeech('Mr', 'row 2 column 2')
+        .call(doCmd('previousRow'))
+        .expectSpeech('title', 'row 1 column 2')
+        .call(doCmd('nextCol'))
+        .expectSpeech('address', 'row 1 column 3')
+        .call(doCmd('nextCol'))
+        .expectSpeech('phone', 'row 1 column 4')
+        .call(doCmd('nextCol'))
+        .expectSpeech('No cell right.')
+        .call(doCmd('previousRow'))
+        .expectSpeech('No cell above.')
+        .call(doCmd('nextRow'))
+        .expectSpeech('212 222 5555', 'row 2 column 4')
+        .call(doCmd('nextRow'))
+        .expectSpeech('No cell below.')
+        .call(doCmd('nextCol'))
+        .expectSpeech('No cell right.')
+        .call(doCmd('previousCol'))
+        .expectSpeech('666 Elm Street', 'row 2 column 3')
+        .call(doCmd('previousCol'))
+        .expectSpeech('Mr', 'row 2 column 2')
+
+        .call(doCmd('goToRowLastCell'))
+        .expectSpeech('212 222 5555', 'row 2 column 4')
+        .call(doCmd('goToRowLastCell'))
+        .expectSpeech('212 222 5555')
+        .call(doCmd('goToRowFirstCell'))
+        .expectSpeech('Dan', 'row 2 column 1')
+        .call(doCmd('goToRowFirstCell'))
+        .expectSpeech('Dan')
+
+        .call(doCmd('goToColFirstCell'))
+    .expectSpeech('name', 'row 1 column 1')
+        .call(doCmd('goToColFirstCell'))
+        .expectSpeech('name')
+        .call(doCmd('goToColLastCell'))
+    .expectSpeech('Dan', 'row 2 column 1')
+        .call(doCmd('goToColLastCell'))
+        .expectSpeech('Dan')
+
+        .call(doCmd('goToLastCell'))
+        .expectSpeech('212 222 5555', 'row 2 column 4')
+        .call(doCmd('goToLastCell'))
+        .expectSpeech('212 222 5555')
+        .call(doCmd('goToFirstCell'))
+        .expectSpeech('name', 'row 1 column 1')
+        .call(doCmd('goToFirstCell'))
+        .expectSpeech('name')
+
+        .replay();
+  });
+});