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

Unified Diff: Source/core/accessibility/AccessibilityObject.cpp

Issue 22331005: Clean up accessibility enums for use in Chromium. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: reupload Created 7 years, 4 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: Source/core/accessibility/AccessibilityObject.cpp
diff --git a/Source/core/accessibility/AccessibilityObject.cpp b/Source/core/accessibility/AccessibilityObject.cpp
index 4ca03134b955dd249dd9e4a05c5bf6c3a5f6433d..725c1542466729347baa9d406b19d16d75eed170 100644
--- a/Source/core/accessibility/AccessibilityObject.cpp
+++ b/Source/core/accessibility/AccessibilityObject.cpp
@@ -57,16 +57,16 @@ struct RoleEntry {
static ARIARoleMap* createARIARoleMap()
{
const RoleEntry roles[] = {
- { "alert", ApplicationAlertRole },
- { "alertdialog", ApplicationAlertDialogRole },
- { "application", LandmarkApplicationRole },
- { "article", DocumentArticleRole },
- { "banner", LandmarkBannerRole },
+ { "alert", AlertRole },
+ { "alertdialog", AlertDialogRole },
+ { "application", ApplicationRole },
+ { "article", ArticleRole },
+ { "banner", BannerRole },
{ "button", ButtonRole },
{ "checkbox", CheckBoxRole },
- { "complementary", LandmarkComplementaryRole },
- { "contentinfo", LandmarkContentInfoRole },
- { "dialog", ApplicationDialogRole },
+ { "complementary", ComplementaryRole },
+ { "contentinfo", ContentInfoRole },
+ { "dialog", DialogRole },
{ "directory", DirectoryRole },
{ "grid", TableRole },
{ "gridcell", CellRole },
@@ -78,41 +78,41 @@ static ARIARoleMap* createARIARoleMap()
{ "group", GroupRole },
{ "heading", HeadingRole },
{ "img", ImageRole },
- { "link", WebCoreLinkRole },
+ { "link", LinkRole },
{ "list", ListRole },
{ "listitem", ListItemRole },
{ "listbox", ListBoxRole },
- { "log", ApplicationLogRole },
+ { "log", LogRole },
// "option" isn't here because it may map to different roles depending on the parent element's role
- { "main", LandmarkMainRole },
- { "marquee", ApplicationMarqueeRole },
- { "math", DocumentMathRole },
+ { "main", MainRole },
+ { "marquee", MarqueeRole },
+ { "math", MathRole },
{ "menu", MenuRole },
{ "menubar", MenuBarRole },
{ "menuitem", MenuItemRole },
{ "menuitemcheckbox", MenuItemRole },
{ "menuitemradio", MenuItemRole },
- { "note", DocumentNoteRole },
- { "navigation", LandmarkNavigationRole },
+ { "note", NoteRole },
+ { "navigation", NavigationRole },
{ "option", ListBoxOptionRole },
{ "presentation", PresentationalRole },
{ "progressbar", ProgressIndicatorRole },
{ "radio", RadioButtonRole },
{ "radiogroup", RadioGroupRole },
- { "region", DocumentRegionRole },
+ { "region", RegionRole },
{ "row", RowRole },
{ "scrollbar", ScrollBarRole },
- { "search", LandmarkSearchRole },
+ { "search", SearchRole },
{ "separator", SplitterRole },
{ "slider", SliderRole },
{ "spinbutton", SpinButtonRole },
- { "status", ApplicationStatusRole },
+ { "status", StatusRole },
{ "tab", TabRole },
{ "tablist", TabListRole },
{ "tabpanel", TabPanelRole },
{ "text", StaticTextRole },
{ "textbox", TextAreaRole },
- { "timer", ApplicationTimerRole },
+ { "timer", TimerRole },
{ "toolbar", ToolbarRole },
{ "tooltip", UserInterfaceTooltipRole },
{ "tree", TreeRole },
@@ -306,7 +306,6 @@ String AccessibilityObject::actionVerb() const
case CheckBoxRole:
return isChecked() ? AXCheckedCheckBoxActionVerb() : AXUncheckedCheckBoxActionVerb();
case LinkRole:
- case WebCoreLinkRole:
return AXLinkActionVerb();
case PopUpButtonRole:
return AXMenuListActionVerb();

Powered by Google App Engine
This is Rietveld 408576698