IAccessible2 state not defined for input type checkbox.

content/browser/accessibility/browser_accessibility_win.cc

 // Copyright (c) 2012 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.
 
 #include "content/browser/accessibility/browser_accessibility_win.h"
 
 #include <UIAutomationClient.h>
 #include <UIAutomationCoreApi.h>
 
 #include "base/strings/string_number_conversions.h"
@@ skipping 3405 matching lines @@
         ia2_role_ = IA2_ROLE_CANVAS;
       } else {
         ia_role_ = ROLE_SYSTEM_GRAPHIC;
       }
       break;
     case ui::AX_ROLE_CELL:
       ia_role_ = ROLE_SYSTEM_CELL;
       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;
       ia2_role_ = IA2_ROLE_COLOR_CHOOSER;
       break;
     case ui::AX_ROLE_COLUMN:
       ia_role_ = ROLE_SYSTEM_COLUMN;
       ia_state_ |= STATE_SYSTEM_READONLY;
       break;
     case ui::AX_ROLE_COLUMN_HEADER:
@@ skipping 367 matching lines @@
   // The role should always be set.
   DCHECK(!role_name_.empty() || ia_role_);
 
   // If we didn't explicitly set the IAccessible2 role, make it the same
   // as the MSAA role.
   if (!ia2_role_)
     ia2_role_ = ia_role_;
 }
 
 }  // namespace content