AX checked state changes

content/browser/accessibility/browser_accessibility_android.cc

Index: content/browser/accessibility/browser_accessibility_android.cc
diff --git a/content/browser/accessibility/browser_accessibility_android.cc b/content/browser/accessibility/browser_accessibility_android.cc
index ba8dbc7ba238237bd21f7d13922295f20be495b8..ee1514764048fde64fb49baa1f0f23a19dc8d5f2 100644
--- a/content/browser/accessibility/browser_accessibility_android.cc
+++ b/content/browser/accessibility/browser_accessibility_android.cc
@@ -144,13 +144,16 @@ bool BrowserAccessibilityAndroid::IsCheckable() const {
       is_aria_pressed_defined) {
     checkable = true;
   }
-  if (HasState(ui::AX_STATE_CHECKED))
+  // TODO(aleventhal) does this ever happen when checkable is not true yet?

[aleventhal, 2017/03/09 17:10:50]

Dominic, we had this extra logic, but I'm not sure what else could be checkable
other than the roles above.

+  if (HasIntAttribute(ui::AX_ATTR_CHECKED_STATE))
     checkable = true;
   return checkable;
 }
 
 bool BrowserAccessibilityAndroid::IsChecked() const {
-  return (HasState(ui::AX_STATE_CHECKED) || HasState(ui::AX_STATE_PRESSED));
+  const int checkedState = GetIntAttribute(ui::AX_ATTR_CHECKED_STATE);
+  return (checkedState == ui::AX_CHECKED_STATE_TRUE ||
+          HasState(ui::AX_STATE_PRESSED));
 }
 
 bool BrowserAccessibilityAndroid::IsClickable() const {