Re-add the line break attribute

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

 // Copyright 2015 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.
 
 /**
  * @fileoverview Processes events related to editing text and emits the
  * appropriate spoken and braille feedback.
  */
 
 goog.provide('editing.TextEditHandler');
@@ skipping 121 matching lines @@
         this.node_.textSelEnd,
         true /* triggered by user */);
     this.changed(textChangeEvent);
     this.outputBraille_();
   },
 
   /** @override */
   getLineIndex: function(charIndex) {
     if (!this.multiline)
       return 0;
-    var breaks = this.getLineBreaks_();
-
-    var computedIndex = 0;
-    for (var index = 0; index < breaks.length; index++) {
-      if (charIndex >= breaks[index] &&
-          (index == (breaks.length - 1) ||
-              charIndex < breaks[index + 1])) {
-        computedIndex = index + 1;
-        break;
-      }
-    }
-    return computedIndex;
+    var breaks = this.node_.lineBreaks || [];
+    var index = 0;
+    while (index < breaks.length && breaks[index] <= charIndex)
+      ++index;
+    return index;
   },
 
   /** @override */
   getLineStart: function(lineIndex) {
     if (!this.multiline || lineIndex == 0)
       return 0;
     var breaks = this.getLineBreaks_();
     return breaks[lineIndex - 1] || this.node_.value.length;
   },
 
   /** @override */
   getLineEnd: function(lineIndex) {
     var breaks = this.getLineBreaks_();
     var value = this.node_.value;
     if (lineIndex >= breaks.length)
       return value.length;
     return breaks[lineIndex] - 1;
   },
 
   /**
    * @return {Array<number>}
    * @private
    */
   getLineBreaks_: function() {
-    if (this.lineBreaks_.length)
-      return this.lineBreaks_;
-
-    // |lineStartOffsets| is sometimes pretty wrong especially when
-    // there's multiple consecutive line breaks.
-
-    // Use Blink's innerText which we plumb through as value.  For
-    // soft line breaks, use line start offsets, but ensure it is
-    // likely to be a soft line wrap.
-
-    var lineStartOffsets = {};
-    this.node_.lineStartOffsets.forEach(function(offset) {
-      lineStartOffsets[offset] = true;
-    });
-    var lineBreaks = [];
-    for (var i = 0; i < this.node_.value.length; i++) {
-      var prev = this.node_.value[i - 1];
-      var next = this.node_.value[i + 1];
-      var cur = this.node_.value[i];
-      if (prev == '\n') {
-        // Hard line break.
-        lineBreaks.push(i);
-      } else if (lineStartOffsets[i]) {
-        // Soft line break.
-        if (prev != '\n' && next != '\n' && cur != '\n')
-          lineBreaks.push(i);
-      }
-    }
-
-    this.lineBreaks_ = lineBreaks;
-
-    return lineBreaks;
+    // node.lineBreaks is undefined when the multiline field has no line
+    // breaks.
+    return this.node_.lineBreaks || [];
   },
 
   /** @private */
   outputBraille_: function() {
     var isFirstLine = false;  // First line in a multiline field.
     var output = new Output();
     var range;
     if (this.multiline) {
       var lineIndex = this.getLineIndex(this.start);
       if (lineIndex == 0) {
@@ skipping 28 matching lines @@
     testNode = testNode.parent;
   } while (testNode);
 
   if (rootFocusedEditable)
     return new TextFieldTextEditHandler(rootFocusedEditable);
 
   return null;
 };
 
 });