[ATK] Adds mapping MenuItemRadioRole to ATK
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 11 Sep 2013 11:31:14 +0000 (11:31 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 11 Sep 2013 11:31:14 +0000 (11:31 +0000)
https://bugs.webkit.org/show_bug.cgi?id=121149

Patch by Krzysztof Czech <k.czech@samsung.com> on 2013-09-11
Reviewed by Mario Sanchez Prada.

Source/WebCore:

Adds mapping MenuItemRadioRole to ATK.

* accessibility/atk/WebKitAccessibleWrapperAtk.cpp:
(atkRole):

Tools:

Adds mapping MenuItemRadioRole to ATK.

* DumpRenderTree/atk/AccessibilityUIElementAtk.cpp:
(roleToString):
* WebKitTestRunner/InjectedBundle/atk/AccessibilityUIElementAtk.cpp:
(WTR::roleToString):

LayoutTests:

Corrected expectations so that they can properly test MenuItemRadioRole.

* platform/efl/accessibility/aria-menubar-menuitems-expected.txt:
* platform/gtk/accessibility/aria-menubar-menuitems-expected.txt:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@155518 268f45cc-cd09-0410-ab3c-d52691b4dbfc

LayoutTests/ChangeLog
LayoutTests/platform/efl/accessibility/aria-menubar-menuitems-expected.txt
LayoutTests/platform/gtk/accessibility/aria-menubar-menuitems-expected.txt
Source/WebCore/ChangeLog
Source/WebCore/accessibility/atk/WebKitAccessibleWrapperAtk.cpp
Tools/ChangeLog
Tools/DumpRenderTree/atk/AccessibilityUIElementAtk.cpp
Tools/WebKitTestRunner/InjectedBundle/atk/AccessibilityUIElementAtk.cpp

index 5341cb5ea3b6712727e05c99f7ef4a8e9ddbcfcc..78b7ad2bbef15395da7957a0f5a34e9f3b12af35 100644 (file)
@@ -1,3 +1,15 @@
+2013-09-11  Krzysztof Czech  <k.czech@samsung.com>
+
+        [ATK] Adds mapping MenuItemRadioRole to ATK
+        https://bugs.webkit.org/show_bug.cgi?id=121149
+
+        Reviewed by Mario Sanchez Prada.
+
+        Corrected expectations so that they can properly test MenuItemRadioRole.
+
+        * platform/efl/accessibility/aria-menubar-menuitems-expected.txt:
+        * platform/gtk/accessibility/aria-menubar-menuitems-expected.txt:
+
 2013-09-11  Chris Fleizach  <cfleizach@apple.com>
 
         AX: AXValue/AXMenuItemMarkChar not defined for menuitemradio (should be false (0) with aria-checked=mixed | undefined)
index 4525c1411bd3f317d2c5a6165c1172c0e6751499..0e5447543c0d1b96d792ec29e60cc5e61fbe8b70 100644 (file)
@@ -16,7 +16,7 @@ AXRole: AXGroup
 PASS menuSubItem1.childrenCount is 0
 AXRole: AXMenu
 PASS menuSubItem2.childrenCount is 1
-AXRole: AXMenuItem
+AXRole: AXRadioMenuItem
 PASS subMenuSubItem.childrenCount is 0
 PASS successfullyParsed is true
 
index 4525c1411bd3f317d2c5a6165c1172c0e6751499..0e5447543c0d1b96d792ec29e60cc5e61fbe8b70 100644 (file)
@@ -16,7 +16,7 @@ AXRole: AXGroup
 PASS menuSubItem1.childrenCount is 0
 AXRole: AXMenu
 PASS menuSubItem2.childrenCount is 1
-AXRole: AXMenuItem
+AXRole: AXRadioMenuItem
 PASS subMenuSubItem.childrenCount is 0
 PASS successfullyParsed is true
 
index 25ba2e6da6599ccc325981a7526df9977775f932..0ded085265f01b68021f6380a67cf646c8cc58c8 100644 (file)
@@ -1,3 +1,15 @@
+2013-09-11  Krzysztof Czech  <k.czech@samsung.com>
+
+        [ATK] Adds mapping MenuItemRadioRole to ATK
+        https://bugs.webkit.org/show_bug.cgi?id=121149
+
+        Reviewed by Mario Sanchez Prada.
+
+        Adds mapping MenuItemRadioRole to ATK.
+
+        * accessibility/atk/WebKitAccessibleWrapperAtk.cpp:
+        (atkRole):
+
 2013-09-10  Mario Sanchez Prada  <mario.prada@samsung.com>
 
         [GTK] Reimplement atk_text_get_text_*_offset for LINE boundaries
index 3e233df71d17e2350e8d154e7cb627358539af01..7cab81b8fcf0357cd5052c5c698245827af6fc1f 100644 (file)
@@ -551,6 +551,8 @@ static AtkRole atkRole(AccessibilityRole role)
     case MenuListOptionRole:
     case MenuItemRole:
         return ATK_ROLE_MENU_ITEM;
+    case MenuItemRadioRole:
+        return ATK_ROLE_RADIO_MENU_ITEM;
     case ColumnRole:
         // return ATK_ROLE_TABLE_COLUMN_HEADER; // Is this right?
         return ATK_ROLE_UNKNOWN; // Matches Mozilla
index 9a4d37ee2677bc41e8b8e8b76e297ab1aae398ff..7897e53756b994286cc15a9dafa17215b340a322 100644 (file)
@@ -1,3 +1,17 @@
+2013-09-11  Krzysztof Czech  <k.czech@samsung.com>
+
+        [ATK] Adds mapping MenuItemRadioRole to ATK
+        https://bugs.webkit.org/show_bug.cgi?id=121149
+
+        Reviewed by Mario Sanchez Prada.
+
+        Adds mapping MenuItemRadioRole to ATK.
+
+        * DumpRenderTree/atk/AccessibilityUIElementAtk.cpp:
+        (roleToString):
+        * WebKitTestRunner/InjectedBundle/atk/AccessibilityUIElementAtk.cpp:
+        (WTR::roleToString):
+
 2013-09-10  Csaba Osztrogon√°c  <ossy@webkit.org>
 
         Unreviewed fix after the unreviewed r155498 to unbreak the failure
index eefca768122f9d56abfd513ac7006b7931f0eec5..7b671a80ad5c745d8b334cb19ef43d643b4816f3 100644 (file)
@@ -158,6 +158,8 @@ static inline const char* roleToString(AtkRole role)
         return "AXButton";
     case ATK_ROLE_RADIO_BUTTON:
         return "AXRadioButton";
+    case ATK_ROLE_RADIO_MENU_ITEM:
+        return "AXRadioMenuItem";
     case ATK_ROLE_ROW_HEADER:
         return "AXRowHeader";
     case ATK_ROLE_RULER:
index 51fe4bdebde98f6516886fbf72b9d405b3da7833..4a4ee07c99796ea1a5dcd1a788eeed6ed8305a1f 100644 (file)
@@ -247,6 +247,8 @@ static const gchar* roleToString(AtkRole role)
         return "AXButton";
     case ATK_ROLE_RADIO_BUTTON:
         return "AXRadioButton";
+    case ATK_ROLE_RADIO_MENU_ITEM:
+        return "AXRadioMenuItem";
     case ATK_ROLE_ROW_HEADER:
         return "AXRowHeader";
     case ATK_ROLE_RULER: