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

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

Issue 6372012: Map a few more WebKit roles to MSAA roles - thanks to Charles... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: Created 9 years, 11 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/accessibility/browser_accessibility_win.cc
===================================================================
--- chrome/browser/accessibility/browser_accessibility_win.cc (revision 72316)
+++ chrome/browser/accessibility/browser_accessibility_win.cc (working copy)
@@ -1324,6 +1324,9 @@
ia_role_ = ROLE_SYSTEM_GROUPING;
ia2_role_ = IA2_ROLE_SECTION;
break;
+ case WebAccessibility::ROLE_BUSY_INDICATOR:
+ ia_role_ = ROLE_SYSTEM_ANIMATION;
+ break;
case WebAccessibility::ROLE_BUTTON:
ia_role_ = ROLE_SYSTEM_PUSHBUTTON;
break;
@@ -1356,6 +1359,9 @@
case WebAccessibility::ROLE_DIALOG:
ia_role_ = ROLE_SYSTEM_DIALOG;
break;
+ case WebAccessibility::ROLE_DISCLOSURE_TRIANGLE:
+ ia_role_ = ROLE_SYSTEM_OUTLINEBUTTON;
+ break;
case WebAccessibility::ROLE_DOCUMENT:
case WebAccessibility::ROLE_WEB_AREA:
ia_role_ = ROLE_SYSTEM_DOCUMENT;
@@ -1376,6 +1382,9 @@
role_name_ = L"div";
ia2_role_ = IA2_ROLE_SECTION;
break;
+ case WebAccessibility::ROLE_GROW_AREA:
+ ia_role_ = ROLE_SYSTEM_GRIP;
+ break;
case WebAccessibility::ROLE_HEADING:
GetAttribute(WebAccessibility::ATTR_HTML_TAG, &role_name_);
ia2_role_ = IA2_ROLE_HEADING;
@@ -1417,6 +1426,9 @@
case WebAccessibility::ROLE_LIST_MARKER:
ia_role_ = ROLE_SYSTEM_LISTITEM;
break;
+ case WebAccessibility::ROLE_MATH:
+ ia_role_ = ROLE_SYSTEM_EQUATION;
+ break;
case WebAccessibility::ROLE_MENU:
case WebAccessibility::ROLE_MENU_BUTTON:
ia_role_ = ROLE_SYSTEM_MENUPOPUP;
@@ -1486,6 +1498,9 @@
case WebAccessibility::ROLE_STATUS:
ia_role_ = ROLE_SYSTEM_STATUSBAR;
break;
+ case WebAccessibility::ROLE_SPLITTER:
+ ia_role_ = ROLE_SYSTEM_SEPARATOR;
+ break;
case WebAccessibility::ROLE_TAB:
ia_role_ = ROLE_SYSTEM_PAGETAB;
break;
@@ -1511,6 +1526,9 @@
ia2_state_ |= IA2_STATE_SINGLE_LINE;
ia2_state_ |= IA2_STATE_EDITABLE;
break;
+ case WebAccessibility::ROLE_TIMER:
+ ia_role_ = ROLE_SYSTEM_CLOCK;
+ break;
case WebAccessibility::ROLE_TOOLBAR:
ia_role_ = ROLE_SYSTEM_TOOLBAR;
break;
@@ -1532,24 +1550,18 @@
// TODO(dmazzoni): figure out the proper MSAA role for all of these.
case WebAccessibility::ROLE_BROWSER:
- case WebAccessibility::ROLE_BUSY_INDICATOR:
case WebAccessibility::ROLE_DIRECTORY:
- case WebAccessibility::ROLE_DISCLOSURE_TRIANGLE:
case WebAccessibility::ROLE_DRAWER:
- case WebAccessibility::ROLE_GROW_AREA:
case WebAccessibility::ROLE_HELP_TAG:
case WebAccessibility::ROLE_IGNORED:
case WebAccessibility::ROLE_INCREMENTOR:
case WebAccessibility::ROLE_LOG:
case WebAccessibility::ROLE_MARQUEE:
- case WebAccessibility::ROLE_MATH:
case WebAccessibility::ROLE_MATTE:
case WebAccessibility::ROLE_RULER_MARKER:
case WebAccessibility::ROLE_SHEET:
case WebAccessibility::ROLE_SLIDER_THUMB:
- case WebAccessibility::ROLE_SPLITTER:
case WebAccessibility::ROLE_SYSTEM_WIDE:
- case WebAccessibility::ROLE_TIMER:
case WebAccessibility::ROLE_VALUE_INDICATOR:
default:
ia_role_ = ROLE_SYSTEM_CLIENT;
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698