Support rich line output in both speech and braille

chrome/browser/resources/chromeos/chromevox/braille/spans.js

Index: chrome/browser/resources/chromeos/chromevox/braille/spans.js
diff --git a/chrome/browser/resources/chromeos/chromevox/braille/spans.js b/chrome/browser/resources/chromeos/chromevox/braille/spans.js
index 525b107e589b97de92251abcac8347790013fcc5..b965c224eaeee7fa4e71b4423d3ddd16c3c8d6f4 100644
--- a/chrome/browser/resources/chromeos/chromevox/braille/spans.js
+++ b/chrome/browser/resources/chromeos/chromevox/braille/spans.js
@@ -8,9 +8,11 @@
  */
 
 goog.provide('cvox.ExtraCellsSpan');
+goog.provide('cvox.FormSpan');
 goog.provide('cvox.ValueSelectionSpan');
 goog.provide('cvox.ValueSpan');
 
+goog.require('cvox.LibLouis.FormType');
 goog.require('Spannable');
 
 /**
@@ -72,3 +74,14 @@ cvox.ExtraCellsSpan = function() {
   /** @type {ArrayBuffer} */
   this.cells = new Uint8Array(0).buffer;
 };
+
+
+/**
+ * Indicates a text form during translation in Liblouis.
+ * @param {cvox.LibLouis.FormType} formType
+ * @constructor
+ */
+cvox.FormSpan = function(formType) {

[dmazzoni, 2017/06/27 21:21:52]

The word "form" has other meanings. formType or typeForm are both used in the
liblouis api so it makes sense to use that, even though they're confusing - but
cvox.FormSpan is our own abstraction so how about naming it something more
verbose and easier to understand.

Maybe cvox.BrailleTextStyleSpan or something like that?

[David Tseng, 2017/06/27 23:52:41]

The spans are really only useful for liblouis, but agreed on readability. Done.

+  /** @type {cvox.LibLouis.FormType} */
+  this.formType = formType;
+};