Adjust output rules related menus

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

 // Copyright 2014 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 Provides output services for ChromeVox.
  */
 
 goog.provide('Output');
 goog.provide('Output.EventType');
@@ skipping 212 matching lines @@
     inherits: 'abstractContainer'
   },
   menu: {
     msgId: 'role_menu',
     earconId: 'LISTBOX',
   },
   menuBar: {
     msgId: 'role_menubar',
   },
   menuItem: {
-    msgId: 'role_menuitem',
-    earconId: 'BUTTON'

[dmazzoni, 2016/06/14 21:46:32]

Did you mean to get rid of the earcon for menuItemCheckBox, below?

[David Tseng, 2016/06/15 22:59:20]

I did; not sure how you feel about it, but output becomes pretty busy with both
checbox and button earcons. I found the button earcon in menus to be a bit much
in general.
Alternatively could try to limit the playing of earcons simultaneously.

+    msgId: 'role_menuitem'
   },
   menuItemCheckBox: {
     msgId: 'role_menuitemcheckbox',
     earconId: 'BUTTON'
   },
   menuItemRadio: {
     msgId: 'role_menuitemradio',
     earconId: 'BUTTON'
   },
   menuListOption: {
@@ skipping 219 matching lines @@
     },
     menu: {
       enter: '$name $role',
       speak: '$name $role @@list_with_items($countChildren(menuItem))'
     },
     menuItem: {
       speak: '$name $role $if($haspopup, @has_submenu) ' +
           '@describe_index($indexInParent, $parentChildCount) ' +
           '$description'
     },
+    menuItemCheckBox: {
+      speak: '$if($checked, $earcon(CHECK_ON), $earcon(CHECK_OFF)) ' +
+            '$name $role $checked $description'
+    },
     menuListOption: {
       speak: '$name @role_menuitem ' +
           '@describe_index($indexInParent, $parentChildCount) $description'
     },
     paragraph: {
       speak: '$descendants'
     },
     popUpButton: {
       speak: '$value $name $role @aria_has_popup ' +
           '$if($collapsed, @aria_expanded_false, @aria_expanded_true) ' +
@@ skipping 1125 matching lines @@
           break;
         }
         earconFinder = earconFinder.parent;
       }
     }
     return null;
   }
 };
 
 });  // goog.scope