More precise use of multiline state

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 102 matching lines @@
     throw Error('Node must have editable state set to true.');
   var start = node.textSelStart;
   var end = node.textSelEnd;
   cvox.ChromeVoxEditableTextBase.call(
       this, node.value || '', Math.min(start, end), Math.max(start, end),
       node.state[StateType.PROTECTED] /**password*/, cvox.ChromeVox.tts);
   /** @override */
   this.multiline = node.state[StateType.MULTILINE] || false;
   /** @type {!AutomationNode} @private */
   this.node_ = node;
-  /** @type {Array<number>} @private */
-  this.lineBreaks_ = [];
 }
 
 AutomationEditableText.prototype = {
   __proto__: cvox.ChromeVoxEditableTextBase.prototype,
 
   /**
    * Called when the text field has been updated.
    */
   onUpdate: function() {
     var newValue = this.node_.value || '';
 
-    if (this.value != newValue)
-      this.lineBreaks_ = [];
-
     var textChangeEvent = new cvox.TextChangeEvent(
         newValue, this.node_.textSelStart || 0, this.node_.textSelEnd || 0,
         true /* triggered by user */);
     this.changed(textChangeEvent);
     this.outputBraille_();
   },
 
   /** @override */
   getLineIndex: function(charIndex) {
     if (!this.multiline)
       return 0;
     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_();
+    if (breaks.length < 1)

[David Tseng, 2017/07/01 01:17:40]

Thanks for the cleanup, but this entire class is pending removal.

If you want to do this, then also make @return {!Array...} for getLineBreaks_
and just check length is 0 here. Length is never < 0.

+      return 0;
     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;
@@ skipping 638 matching lines @@
    * @return {boolean}
    */
   isSameLineAndSelection: function(otherLine) {
     return this.isSameLine(otherLine) &&
         this.startOffset == otherLine.startOffset &&
         this.endOffset == otherLine.endOffset;
   }
 };
 
 });