IAccessible2 state not defined for input type checkbox.

content/browser/accessibility/accessibility_tree_formatter_utils_win.cc

Index: content/browser/accessibility/accessibility_tree_formatter_utils_win.cc
diff --git a/content/browser/accessibility/accessibility_tree_formatter_utils_win.cc b/content/browser/accessibility/accessibility_tree_formatter_utils_win.cc
index 5e1d97351e1a6c3910116957c646ef987fd72b7d..02159bca1adb4f6d267f49236c2f3d6efd98be6d 100644
--- a/content/browser/accessibility/accessibility_tree_formatter_utils_win.cc
+++ b/content/browser/accessibility/accessibility_tree_formatter_utils_win.cc
@@ -197,6 +197,7 @@ AccessibilityRoleStateMap::AccessibilityRoleStateMap() {
   // IAccessible2 states.
   IA2_STATE_MAP(IA2_STATE_ACTIVE)
   IA2_STATE_MAP(IA2_STATE_ARMED)
+  IA2_STATE_MAP(IA2_STATE_CHECKABLE)

[shreeramk, 2014/09/26 18:42:16]

IA2_STATE_CHECKABLE was handled here but how to handle following IA2 states -
STATE_SYSTEM_CHECKED and STATE_SYSTEM_MIXED??

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

Those are defined above:

IA_STATE_MAP(CHECKED)
IA_STATE_MAP(MIXED)

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

But according to specs STATE_SYSTEM_CHECKED and STATE_SYSTEM_MIXED has to be
defined for IA2_STATE.

[dmazzoni, 2014/09/29 05:44:06]

No, you're misunderstanding the spec.

IAccessible2 is an extension of MSAA/IAccessible. It doesn't replace it, it
extends it.

The spec has a column for what a MSAA+IAccessible2 browser should do. That's the
column we should follow for Chrome Windows support. Anything in that column
that's an IA2-only role or attribute, we return from IA2 functions (using:
ia2_role_ and ia2_state_). Anything that's a role or attribute from MSAA/IA, we
return from the MSAA/IA functions.

In this case, STATE_SYSTEM_CHECKED and STATE_SYSTEM_MIXED are already part of
MSAA/IA, so we set them in ia_state_ and nothing IA2-specific is needed.

   IA2_STATE_MAP(IA2_STATE_DEFUNCT)
   IA2_STATE_MAP(IA2_STATE_EDITABLE)
   IA2_STATE_MAP(IA2_STATE_ICONIFIED)