IAccessible2 state not defined for input type checkbox.

content/browser/accessibility/browser_accessibility_win.cc

Index: content/browser/accessibility/browser_accessibility_win.cc
diff --git a/content/browser/accessibility/browser_accessibility_win.cc b/content/browser/accessibility/browser_accessibility_win.cc
index ff239430f396c979c0644cf9aabd8e6b5a260ec2..1640a94b0efaaa4541fc884b98d3c8c73de6c9f6 100644
--- a/content/browser/accessibility/browser_accessibility_win.cc
+++ b/content/browser/accessibility/browser_accessibility_win.cc
@@ -3423,6 +3423,15 @@ void BrowserAccessibilityWin::InitRoleAndState() {
       break;
     case ui::AX_ROLE_CHECK_BOX:
       ia_role_ = ROLE_SYSTEM_CHECKBUTTON;
+      bool is_aria_checked_defined;
+      bool is_mixed_check;
+      if (GetAriaTristate("aria-checked",
+        &is_aria_checked_defined, &is_mixed_check)) {

[dmazzoni, 2014/09/26 18:52:43]

nit: indent 4 chars for line continuation

+        ia_state_ = STATE_SYSTEM_CHECKED;

[shreeramk, 2014/09/26 19:14:43]

I think I made a mistake here in uploading the patch. (instead of ia2_state_, I
used ia_state_.

Because According to specs following states STATE_SYSTEM_CHECKED and
STATE_SYSTEM_MIXED has to be exposed through IA2_STATE.

But, we won't be able to expose those states as IA2_STATE, since these are not
defined using IA2_STATE_MAP macro.

Rather it has been defined using IA_STATE_MAP macro.

+        ia2_state_ = IA2_STATE_CHECKABLE;
+        if (is_mixed_check)
+          ia_state_ = STATE_SYSTEM_MIXED;

[dmazzoni, 2014/09/26 18:52:43]

This should be an |= to preserve any other state flags

+      }
       break;
     case ui::AX_ROLE_COLOR_WELL:
       ia_role_ = ROLE_SYSTEM_CLIENT;