Improve braille related message descriptions and clean up message handling in Chromevox.

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 7c9ee8e6c40c1c52f44345412a7ffd33de1c0220..451a93bee3c42d47e653d61b18fcdfeb2f7adcfa 100644
--- a/chrome/browser/resources/chromeos/chromevox/cvox2/background/output.js
+++ b/chrome/browser/resources/chromeos/chromevox/cvox2/background/output.js
@@ -111,18 +111,18 @@ Output.ROLE_INFO_ = {
     inherits: 'abstractContainer'
   },
   button: {
-    msgId: 'tag_button',
+    msgId: 'aria_role_button',
     earconId: 'BUTTON'
   },
   buttonDropDown: {
-    msgId: 'tag_button',
+    msgId: 'aria_role_button',
     earconId: 'BUTTON'
   },
   cell: {
     msgId: 'aria_role_gridcell'
   },
   checkBox: {
-    msgId: 'input_type_checkbox'
+    msgId: 'aria_role_checkbox'
   },
   columnHeader: {
     msgId: 'aria_role_columnheader',
@@ -148,7 +148,7 @@ Output.ROLE_INFO_ = {
     inherits: 'abstractContainer'
   },
   dialog: {
-    msgId: 'dialog'
+    msgId: 'aria_role_dialog'
   },
   directory: {
     msgId: 'aria_role_directory',
@@ -238,11 +238,11 @@ Output.ROLE_INFO_ = {
     inherits: 'abstractContainer'
   },
   popUpButton: {
-    msgId: 'tag_button',
+    msgId: 'aria_role_button',
     earcon: 'LISTBOX'
   },
   radioButton: {
-    msgId: 'input_type_radio'
+    msgId: 'aria_role_radio'
   },
   radioGroup: {
     msgId: 'aria_role_radiogroup',
@@ -363,11 +363,10 @@ Output.STATE_INFO_ = {
  */
 Output.INPUT_TYPE_MESSAGE_IDS_ = {
   'email': 'input_type_email',
-  'file': 'input_type_file',
   'number': 'input_type_number',
   'password': 'input_type_password',
   'search': 'input_type_search',
-  'tel': 'input_type_tel',
+  'tel': 'input_type_number',

[dmazzoni, 2015/09/25 16:23:35]

Is this intentional? I think we should distinguish a telephone number field from
a number field

   'text': 'input_type_text',
   'url': 'input_type_url',
 };
@@ -895,9 +894,9 @@ Output.prototype = {
           var info = Output.ROLE_INFO_[node.role];
           if (info) {
             if (this.formatOptions_.braille)
-              msg = cvox.ChromeVox.msgs.getMsg(info.msgId + '_brl');
+              msg = Msgs.getMsg(info.msgId + '_brl');
             else
-              msg = cvox.ChromeVox.msgs.getMsg(info.msgId);
+              msg = Msgs.getMsg(info.msgId);
           } else {
             console.error('Missing role info for ' + node.role);
           }
@@ -910,7 +909,7 @@ Output.prototype = {
               'input_type_text';
           if (this.formatOptions_.braille)
             msgId = msgId + '_brl';
-          this.append_(buff, cvox.ChromeVox.msgs.getMsg(msgId), options);
+          this.append_(buff, Msgs.getMsg(msgId), options);
         } else if (token == 'tableRowIndex' ||
             token == 'tableCellColumnIndex') {
           var value = node[token];
@@ -942,7 +941,7 @@ Output.prototype = {
           var msgId =
               this.formatOptions_.braille ? resolvedInfo.msgId + '_brl' :
               resolvedInfo.msgId;
-          var msg = cvox.ChromeVox.msgs.getMsg(msgId);
+          var msg = Msgs.getMsg(msgId);
           this.append_(buff, msg, options);
         } else if (tree.firstChild) {
           // Custom functions.
@@ -999,10 +998,10 @@ Output.prototype = {
             curArg = curArg.nextSibling;
           }
         }
-        var msg = cvox.ChromeVox.msgs.getMsg(msgId, msgArgs);
+        var msg = Msgs.getMsg(msgId, msgArgs);
         try {
           if (this.formatOptions_.braille)
-            msg = cvox.ChromeVox.msgs.getMsg(msgId + '_brl', msgArgs) || msg;
+            msg = Msgs.getMsg(msgId + '_brl', msgArgs) || msg;
         } catch(e) {}
 
         if (!msg) {