Chromium Code Reviews| Index: chrome/browser/resources/chromeos/chromevox/cvox2/background/desktop_automation_handler.js |
| diff --git a/chrome/browser/resources/chromeos/chromevox/cvox2/background/desktop_automation_handler.js b/chrome/browser/resources/chromeos/chromevox/cvox2/background/desktop_automation_handler.js |
| index d5da5072a72ead8998c936abee769ddb2527936f..bf9cd512afeb7718f283a0cd5b2aa1ffb51478c1 100644 |
| --- a/chrome/browser/resources/chromeos/chromevox/cvox2/background/desktop_automation_handler.js |
| +++ b/chrome/browser/resources/chromeos/chromevox/cvox2/background/desktop_automation_handler.js |
| @@ -45,7 +45,7 @@ DesktopAutomationHandler = function(node) { |
| 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); |
| @@ -202,6 +202,20 @@ DesktopAutomationHandler.prototype = { |
| }, |
| /** |
| + * 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 th
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 |
| */ |