Chromevox2: make sure the braille cursor is shown for empty text fields.

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

Index: chrome/browser/resources/chromeos/chromevox/cvox2/background/output.js
diff --git a/chrome/browser/resources/chromeos/chromevox/cvox2/background/output.js b/chrome/browser/resources/chromeos/chromevox/cvox2/background/output.js
index fd65f4b02db4563eadb8ac6bce258614c390a01f..ebe8659023fbab59088e6646d52aa82a58eff09c 100644
--- a/chrome/browser/resources/chromeos/chromevox/cvox2/background/output.js
+++ b/chrome/browser/resources/chromeos/chromevox/cvox2/background/output.js
@@ -331,19 +331,17 @@ Output.prototype = {
         this.brailleBuffer_.getSpanInstanceOf(Output.SelectionSpan);
     var startIndex = -1, endIndex = -1;
     if (selSpan) {
-      var valueStart = this.brailleBuffer_.getSpanStart(selSpan);
-      var valueEnd = this.brailleBuffer_.getSpanEnd(selSpan);
-      if (valueStart === undefined || valueEnd === undefined) {
-        valueStart = -1;
-        valueEnd = -1;
-      } else {
-        startIndex = valueStart + selSpan.startIndex;
-        endIndex = valueStart + selSpan.endIndex;
-        this.brailleBuffer_.setSpan(new cvox.ValueSpan(valueStart),
-                                    valueStart, valueEnd);
-        this.brailleBuffer_.setSpan(new cvox.ValueSelectionSpan(),
-                                    startIndex, endIndex);
-      }
+      // Casts ok, since the span is known to be in the spannable.
+      var valueStart =
+          /** @type {number} */ (this.brailleBuffer_.getSpanStart(selSpan));
+      var valueEnd =
+          /** @type {number} */ (this.brailleBuffer_.getSpanEnd(selSpan));
+      startIndex = valueStart + selSpan.startIndex;
+      endIndex = valueStart + selSpan.endIndex;
+      this.brailleBuffer_.setSpan(new cvox.ValueSpan(0),
+                                  valueStart, valueEnd);
+      this.brailleBuffer_.setSpan(new cvox.ValueSelectionSpan(),
+                                  startIndex, endIndex);
     }
 
     var output = new cvox.NavBraille({
@@ -432,7 +430,7 @@ Output.prototype = {
           this.addToSpannable_(buff, node.role, options);
         } else if (token == 'value') {
           var text = node.attributes.value;
-          if (text) {
+          if (text !== undefined) {
             var offset = buff.getLength();
             if (node.attributes.textSelStart !== undefined) {
               options.annotation = new Output.SelectionSpan(