diff options
| author | shreeram.k <shreeram.k@samsung.com> | 2014-10-08 21:45:14 -0700 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2014-10-09 04:45:46 +0000 |
| commit | 93f1e4e35afa32aefe0ad065e7fe166ca25a9f22 (patch) | |
| tree | 4b57cb367f0bd43bc230b588a18bd476797b0abf /content/browser/accessibility/browser_accessibility_win.cc | |
| parent | 36b7fc7f8b05ea627873e58a162c1c26784e472d (diff) | |
| download | chromium_src-93f1e4e35afa32aefe0ad065e7fe166ca25a9f22.zip chromium_src-93f1e4e35afa32aefe0ad065e7fe166ca25a9f22.tar.gz chromium_src-93f1e4e35afa32aefe0ad065e7fe166ca25a9f22.tar.bz2 | |
Expose aria role menuitemcheckbox and menuitemradio with correct IA2 role.(Chromium)
According to specs(http://www.w3.org/WAI/PF/aria-implementation/)
aria role menuitemcheckbox should have IA2 role as IA2_ROLE_CHECK_MENU_ITEM
aria role menuitemradio should have IA2 role as IA2_ROLE_RADIO_MENU_ITEM
This CL depends on https://codereview.chromium.org/629003003/ and https://codereview.chromium.org/631113002/
BUG=169569, 420605, 169573
TBR=kalman
Review URL: https://codereview.chromium.org/633043002
Cr-Commit-Position: refs/heads/master@{#298780}
Diffstat (limited to 'content/browser/accessibility/browser_accessibility_win.cc')
| -rw-r--r-- | content/browser/accessibility/browser_accessibility_win.cc | 10 |
1 files changed, 10 insertions, 0 deletions
diff --git a/content/browser/accessibility/browser_accessibility_win.cc b/content/browser/accessibility/browser_accessibility_win.cc index 355b188..e7b206c 100644 --- a/content/browser/accessibility/browser_accessibility_win.cc +++ b/content/browser/accessibility/browser_accessibility_win.cc @@ -2928,6 +2928,8 @@ void BrowserAccessibilityWin::OnDataChanged() { if (ia_role_ == ROLE_SYSTEM_CHECKBUTTON || ia_role_ == ROLE_SYSTEM_RADIOBUTTON || + ia2_role_ == IA2_ROLE_CHECK_MENU_ITEM || + ia2_role_ == IA2_ROLE_RADIO_MENU_ITEM || ia2_role_ == IA2_ROLE_TOGGLE_BUTTON) { ia2_attributes_.push_back(L"checkable:true"); } @@ -3615,6 +3617,14 @@ void BrowserAccessibilityWin::InitRoleAndState() { case ui::AX_ROLE_MENU_ITEM: ia_role_ = ROLE_SYSTEM_MENUITEM; break; + case ui::AX_ROLE_MENU_ITEM_CHECK_BOX: + ia_role_ = ROLE_SYSTEM_MENUITEM; + ia2_role_ = IA2_ROLE_CHECK_MENU_ITEM; + break; + case ui::AX_ROLE_MENU_ITEM_RADIO: + ia_role_ = ROLE_SYSTEM_MENUITEM; + ia2_role_ = IA2_ROLE_RADIO_MENU_ITEM; + break; case ui::AX_ROLE_MENU_LIST_POPUP: ia_role_ = ROLE_SYSTEM_CLIENT; break; |