Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(17)

Unified Diff: content/browser/accessibility/browser_accessibility_win.cc

Issue 1030333002: Chromium side Implementation of new roles added in ARIA 1.1 draft. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Updating failed expectations Created 5 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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 228fb6307f9d7aad9cc20e30f17abbe85a1ba6bd..5c318b29e1c7de1041fc3a12a90404645cc80052 100644
--- a/content/browser/accessibility/browser_accessibility_win.cc
+++ b/content/browser/accessibility/browser_accessibility_win.cc
@@ -3861,10 +3861,6 @@ void BrowserAccessibilityWin::InitRoleAndState() {
ia_role = ROLE_SYSTEM_TEXT;
ia2_role = IA2_ROLE_LABEL;
break;
- case ui::AX_ROLE_SEARCH:
- ia_role = ROLE_SYSTEM_GROUPING;
- ia2_role = IA2_ROLE_SECTION;
- break;
case ui::AX_ROLE_LINK:
ia_role = ROLE_SYSTEM_LINK;
ia_state |= STATE_SYSTEM_LINKED;
@@ -4002,6 +3998,10 @@ void BrowserAccessibilityWin::InitRoleAndState() {
case ui::AX_ROLE_SCROLL_BAR:
ia_role = ROLE_SYSTEM_SCROLLBAR;
break;
+ case ui::AX_ROLE_SEARCH:
+ ia_role = ROLE_SYSTEM_GROUPING;
+ ia2_role = IA2_ROLE_SECTION;
+ break;
case ui::AX_ROLE_SLIDER:
ia_role = ROLE_SYSTEM_SLIDER;
break;
@@ -4025,6 +4025,10 @@ void BrowserAccessibilityWin::InitRoleAndState() {
case ui::AX_ROLE_SVG_ROOT:
ia_role = ROLE_SYSTEM_GRAPHIC;
break;
+ case ui::AX_ROLE_SWITCH:
+ role_name = L"switch";
+ ia2_role = IA2_ROLE_TOGGLE_BUTTON;
+ break;
case ui::AX_ROLE_TAB:
ia_role = ROLE_SYSTEM_PAGETAB;
break;
@@ -4060,6 +4064,7 @@ void BrowserAccessibilityWin::InitRoleAndState() {
ia2_state |= IA2_STATE_SELECTABLE_TEXT;
break;
case ui::AX_ROLE_TEXT_FIELD:
+ case ui::AX_ROLE_SEARCH_BOX:
ia_role = ROLE_SYSTEM_TEXT;
ia2_state |= IA2_STATE_SINGLE_LINE;
ia2_state |= IA2_STATE_EDITABLE;

Powered by Google App Engine
This is Rietveld 408576698