Implement computedRole and computedName (behind a flag)

Source/modules/accessibility/AXObject.cpp

 /*
  * Copyright (C) 2008, 2009, 2011 Apple Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  *
  * 1.  Redistributions of source code must retain the above copyright
  *     notice, this list of conditions and the following disclaimer.
  * 2.  Redistributions in binary form must reproduce the above copyright
@@ skipping 31 matching lines @@
 #include "platform/text/PlatformLocale.h"
 #include "wtf/StdLibExtras.h"
 #include "wtf/text/WTFString.h"
 
 using blink::WebLocalizedString;
 
 namespace blink {
 
 using namespace HTMLNames;
 
+namespace {
 typedef HashMap<String, AccessibilityRole, CaseFoldingHash> ARIARoleMap;
 
 struct RoleEntry {
     String ariaRole;
     AccessibilityRole webcoreRole;
 };
 
-static ARIARoleMap* createARIARoleMap()
+void fillRoles(Vector<RoleEntry>* rolesVector)
 {
-    const RoleEntry roles[] = {
+    RoleEntry roles[] = {
         { "alert", AlertRole },
         { "alertdialog", AlertDialogRole },
         { "application", ApplicationRole },
         { "article", ArticleRole },
         { "banner", BannerRole },
         { "button", ButtonRole },
         { "checkbox", CheckBoxRole },
         { "complementary", ComplementaryRole },
         { "contentinfo", ContentInfoRole },
         { "dialog", DialogRole },
         { "directory", DirectoryRole },
         { "grid", GridRole },
         { "gridcell", CellRole },
         { "columnheader", ColumnHeaderRole },
         { "combobox", ComboBoxRole },
         { "definition", DefinitionRole },
         { "document", DocumentRole },
         { "rowheader", RowHeaderRole },
         { "form", FormRole },
         { "group", GroupRole },
         { "heading", HeadingRole },
         { "img", ImageRole },
         { "link", LinkRole },
         { "list", ListRole },
         { "listitem", ListItemRole },
         { "listbox", ListBoxRole },
         { "log", LogRole },
         // "option" isn't here because it may map to different roles depending on the parent element's role

[dmazzoni, 2014/12/12 00:41:48]

Maybe try to handle this role as a special case?

[aboxhall, 2014/12/13 01:38:36]

This comment is actually a lie - option is present a few lines down.

         { "main", MainRole },
         { "marquee", MarqueeRole },
         { "math", MathRole },
         { "menu", MenuRole },
         { "menubar", MenuBarRole },
         { "menuitem", MenuItemRole },
         { "menuitemcheckbox", MenuItemCheckBoxRole },
         { "menuitemradio", MenuItemRadioRole },
         { "note", NoteRole },
         { "navigation", NavigationRole },
@@ skipping 16 matching lines @@
         { "tabpanel", TabPanelRole },
         { "text", StaticTextRole },
         { "textbox", TextAreaRole },
         { "timer", TimerRole },
         { "toolbar", ToolbarRole },
         { "tooltip", UserInterfaceTooltipRole },
         { "tree", TreeRole },
         { "treegrid", TreeGridRole },
         { "treeitem", TreeItemRole }
     };
+    size_t numRoles = WTF_ARRAY_LENGTH(roles);
+    rolesVector->resize(numRoles);
+    for (size_t i = 0; i < numRoles; ++i)
+        rolesVector->at(i) = roles[i];
+}
+
+static ARIARoleMap* createARIARoleMap()
+{
     ARIARoleMap* roleMap = new ARIARoleMap;
 
-    for (size_t i = 0; i < WTF_ARRAY_LENGTH(roles); ++i)
+    Vector<RoleEntry> roles;
+    fillRoles(&roles);
+    for (size_t i = 0; i < roles.size(); ++i)
         roleMap->set(roles[i].ariaRole, roles[i].webcoreRole);
     return roleMap;
 }
 
+static void fillRoleNameVector(Vector<AtomicString>* roleNameVector)
+{
+    roleNameVector->resize(NumRoles);
+    for (int i = 0; i < NumRoles; i++) {
+        roleNameVector->at(i) = AtomicString();
+    }
+
+    Vector<RoleEntry> roles;
+
+    fillRoles(&roles);
+    for (size_t i = 0; i < roles.size(); ++i) {
+        roleNameVector->insert(roles[i].webcoreRole, AtomicString(roles[i].ariaRole));
+    }
+}
+
+} // namespace
+
 AXObject::AXObject(AXObjectCacheImpl* axObjectCache)
     : m_id(0)
     , m_haveChildren(false)
     , m_role(UnknownRole)
     , m_lastKnownIsIgnoredValue(DefaultBehavior)
     , m_detached(false)
     , m_parent(0)
     , m_lastModificationCount(-1)
     , m_cachedIsIgnored(false)
     , m_cachedLiveRegionRoot(0)
@@ skipping 804 matching lines @@
     if (ariaPressedIsPresent())
         return ToggleButtonRole;
     if (ariaHasPopup())
         return PopUpButtonRole;
     // We don't contemplate RadioButtonRole, as it depends on the input
     // type.
 
     return ButtonRole;
 }
 
+const AtomicString& AXObject::roleName(const AccessibilityRole role)
+{
+    static Vector<AtomicString> roleNameVector;
+    if (roleNameVector.size() == 0)
+        fillRoleNameVector(&roleNameVector);
+
+    return roleNameVector[role];

[dmazzoni, 2014/12/12 00:41:48]

Add a check/assert that this is in-bounds

[aboxhall, 2014/12/13 01:38:36]

Done.

+}
+
 } // namespace blink