Expand EditableLine to include non-inline text box leafs

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

Index: chrome/browser/resources/chromeos/chromevox/cvox2/background/editing_test.extjs
diff --git a/chrome/browser/resources/chromeos/chromevox/cvox2/background/editing_test.extjs b/chrome/browser/resources/chromeos/chromevox/cvox2/background/editing_test.extjs
index 6bfb586ce35c6eb8a0fd9e5eaad91991af31c3b6..7fc8327e0be31f9dfd490b6c2cd3da58eec7f1f7 100644
--- a/chrome/browser/resources/chromeos/chromevox/cvox2/background/editing_test.extjs
+++ b/chrome/browser/resources/chromeos/chromevox/cvox2/background/editing_test.extjs
@@ -484,6 +484,71 @@ TEST_F('EditingTest', 'RichTextExtendByCharacter', function() {
   });
 });
 
+TEST_F('EditingTest', 'RichTextImageByCharacter', function() {
+  var mockFeedback = this.createMockFeedback();
+  this.runWithLoadedTree(function() {/*!
+    <p id="go" contenteditable>
+      <img alt="dog"></img> is a <img alt="cat"</img> test
+    </p>
+    <script>
+      document.getElementById('go').addEventListener('click', function() {
+        var sel = getSelection();
+        sel.modify('move', 'forward', 'character');
+      }, true);
+    </script>
+  */}, function(root) {
+    var input = root.find({role: RoleType.PARAGRAPH});
+    var moveByChar = input.doDefault.bind(input);
+
+    this.listenOnce(input, 'focus', function() {
+      var lineText = 'dog is a cat test';
+      mockFeedback.call(moveByChar)
+          // Note that this isn't quite correct. This should be the second
+          // character which is a space ' '.
+          .expectSpeech('dog', 'Image')
+          // Ditto.
+          .expectBraille(lineText, {startIndex: 0, endIndex: 0})
+          .call(moveByChar)
+          .expectSpeech('i')
+          .expectBraille(lineText, {startIndex: 4, endIndex: 4})
+          .call(moveByChar)
+          .expectSpeech('s')
+          .expectBraille(lineText, {startIndex: 5, endIndex: 5})
+          .call(moveByChar)
+          .expectSpeech(' ')
+          .expectBraille(lineText, {startIndex: 6, endIndex: 6})
+          .call(moveByChar)
+          .expectSpeech('a')
+          .expectBraille(lineText, {startIndex: 7, endIndex: 7})
+          .call(moveByChar)
+          .expectSpeech(' ')
+          .expectBraille(lineText, {startIndex: 8, endIndex: 8})
+          .call(moveByChar)
+
+          // This is technically wrong because we're actually over the entire
+          // image. This is broken because of bad node offsets.
+          .expectSpeech('c')
+          .expectBraille(lineText, {startIndex: 9, endIndex: 9})
+
+          // Unfortunately, the node offset being wrong here means there's no
+          // output for this character move which gets us past the cat image
+          // node.
+          .call(moveByChar)
+
+          // And, we're back (skipped the space).
+          .call(moveByChar)
+          // Unfortunately, due to the above, we think we're on a new line and
+          // read it all.
+          .expectSpeech('dog', 'Image', ' is a ', 'cat', 'Image', ' test')
+          // But, we're at least on the right caret position.
+          .expectBraille(lineText, {startIndex: 13, endIndex: 13})
+
+          .replay();
+    });
+    input.focus();
+  });
+});
+
 TEST_F('EditingTest', 'EditableLineOneStaticText', function() {
   this.runWithLoadedTree(function() {/*!
     <p contenteditable style="word-spacing:100000px">this is a test</p>