Only process checked state changed for specific roles.

chrome/browser/resources/chromeos/chromevox/cvox2/background/desktop_automation_handler.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 Handles automation from a desktop automation node.
  */
 
 goog.provide('DesktopAutomationHandler');
 
@@ skipping 27 matching lines @@
    * The last time we handled a value changed event.
    * @type {!Date}
    * @private
    */
   this.lastValueChanged_ = new Date(0);
 
   var e = EventType;
   this.addListener_(e.activedescendantchanged, this.onActiveDescendantChanged);
   this.addListener_(e.alert, this.onAlert);
   this.addListener_(e.ariaAttributeChanged, this.onEventIfInRange);
-  this.addListener_(e.checkedStateChanged, this.onEventIfInRange);
+  this.addListener_(e.checkedStateChanged, this.onCheckedStateChanged);
   this.addListener_(e.focus, this.onFocus);
   this.addListener_(e.hover, this.onHover);
   this.addListener_(e.loadComplete, this.onLoadComplete);
   this.addListener_(e.menuEnd, this.onMenuEnd);
   this.addListener_(e.menuListItemSelected, this.onEventIfSelected);
   this.addListener_(e.menuStart, this.onMenuStart);
   this.addListener_(e.scrollPositionChanged, this.onScrollPositionChanged);
   this.addListener_(e.selection, this.onSelection);
   this.addListener_(e.textChanged, this.onTextChanged);
   this.addListener_(e.textSelectionChanged, this.onTextSelectionChanged);
@@ skipping 136 matching lines @@
         ChromeVoxState.instance.mode === ChromeVoxMode.CLASSIC) {
       return;
     }
 
     var range = cursors.Range.fromNode(node);
 
     new Output().withSpeechAndBraille(range, null, evt.type).go();
   },
 
   /**
+   * Provides all feedback once a checked state changed event fires.
+   * @param {!AutomationEvent} evt
+   */
+  onCheckedStateChanged: function(evt) {
+    if (!AutomationPredicate.checkable(evt.target))
+      return;
+    this.onEventDefault(

[dmazzoni, 2016/06/22 05:20:01]

Should this maybe be onEventIfInRange?

I don't think you want it announcing check boxes changing when they're not
focused, most of the time.

[David Tseng, 2016/06/28 19:36:55]

Done.

+        new chrome.automation.AutomationEvent(
+            EventType.checkedStateChanged, evt.target));
+
+
+  },
+
+  /**
    * Provides all feedback once a focus event fires.
    * @param {!AutomationEvent} evt
    */
   onFocus: function(evt) {
     // Invalidate any previous editable text handler state.
     this.textEditHandler_ = null;
 
     var node = evt.target;
 
     // Discard focus events on embeddedObject and client nodes.
@@ skipping 197 matching lines @@
   if (cvox.ChromeVox.isMac)
     return;
   chrome.automation.getDesktop(function(desktop) {
     global.desktopAutomationHandler = new DesktopAutomationHandler(desktop);
   });
 };
 
 DesktopAutomationHandler.init_();
 
 });  // goog.scope