AX checked state changes

chrome/renderer/resources/extensions/automation/automation_node.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.
 
 var AutomationEvent = require('automationEvent').AutomationEvent;
 var automationInternal =
     require('binding').Binding.create('automationInternal').generate();
 var exceptionHandler = require('uncaught_exception_handler');
 var IsInteractPermitted =
     requireNative('automationInternal').IsInteractPermitted;
@@ skipping 117 matching lines @@
  * @param {number} axTreeID The id of the accessibility tree.
  * @param {number} nodeID The id of a node.
  * @return {?Object} An object with a string key for every state flag set,
  *     or undefined if the tree or node or node parent wasn't found.
  */
 var GetState = requireNative('automationInternal').GetState;
 
 /**
  * @param {number} axTreeID The id of the accessibility tree.
  * @param {number} nodeID The id of a node.
+ * @return {string} The checked state, as undefined, "true", "false" or "mixed".
+ */
+var GetChecked = requireNative('automationInternal').GetChecked;
+
+/**
+ * @param {number} axTreeID The id of the accessibility tree.
+ * @param {number} nodeID The id of a node.
  * @return {string} The role of the node, or undefined if the tree or
  *     node wasn't found.
  */
 var GetRole = requireNative('automationInternal').GetRole;
 
 /**
  * @param {number} axTreeID The id of the accessibility tree.
  * @param {number} nodeID The id of a node.
  * @return {?automation.Rect} The location of the node, or undefined if
  *     the tree or node wasn't found.
@@ skipping 124 matching lines @@
   },
 
   get htmlAttributes() {
     return GetHtmlAttributes(this.treeID, this.id) || {};
   },
 
   get state() {
     return GetState(this.treeID, this.id) || {};
   },
 
   get role() {

[dmazzoni, 2017/03/07 17:35:36]

You need to implement checked() here, something like:

get checked() {
  return GetChecked(this.treeID, this.id);
}

     return GetRole(this.treeID, this.id);
   },
 
   get location() {
     return GetLocation(this.treeID, this.id);
   },
 
   boundsForRange: function(startIndex, endIndex) {
     return GetBoundsForRange(this.treeID, this.id, startIndex, endIndex);
   },
@@ skipping 396 matching lines @@
     'liveRelevant',
     'liveStatus',
     'name',
     'placeholder',
     'textInputType',
     'url',
     'value'];
 
 var boolAttributes = [
     'ariaReadonly',
-    'buttonMixed',
     'containerLiveAtomic',
     'containerLiveBusy',
     'liveAtomic',
     'liveBusy'];
 
 var intAttributes = [
     'backgroundColor',
     'color',
     'colorValue',
     'hierarchicalLevel',
@@ skipping 384 matching lines @@
     'boundsForRange',
   ],
   readonly: $Array.concat(publicAttributes, [
       'parent',
       'firstChild',
       'lastChild',
       'children',
       'previousSibling',
       'nextSibling',
       'isRootNode',
       'role',

[dmazzoni, 2017/03/07 17:35:36]

You also need to add 'checked' to this list so that the
attribute is actually exposed

       'state',
       'location',
       'indexInParent',
       'lineStartOffsets',
       'root',
       'htmlAttributes',
       'nameFrom',
   ]),
 });
 
@@ skipping 24 matching lines @@
 utils.defineProperty(AutomationRootNode, 'getOrCreate', function(treeID) {
   return AutomationRootNodeImpl.getOrCreate(treeID);
 });
 
 utils.defineProperty(AutomationRootNode, 'destroy', function(treeID) {
   AutomationRootNodeImpl.destroy(treeID);
 });
 
 exports.$set('AutomationNode', AutomationNode);
 exports.$set('AutomationRootNode', AutomationRootNode);