AX: [ATK] Video and audio elements should be exposed using ATK's video and audio...
authorjdiggs@igalia.com <jdiggs@igalia.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sun, 16 Nov 2014 07:32:08 +0000 (07:32 +0000)
committerjdiggs@igalia.com <jdiggs@igalia.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sun, 16 Nov 2014 07:32:08 +0000 (07:32 +0000)
https://bugs.webkit.org/show_bug.cgi?id=138767

Reviewed by Chris Fleizach.

Source/WebCore:

Map AudioRole and VideoRole to ATK_ROLE_AUDIO and ATK_ROLE_VIDEO
respectively, falling back on the existing mapping to ATK_ROLE_EMBEDDED
in environments which have an older version of ATK.

No new tests are required. Instead, existing expectations were updated
to reflect the new mapping.

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

Tools:

Add mappings for ATK_ROLE_AUDIO and ATK_ROLE_VIDEO to roleToString().

* WebKitTestRunner/InjectedBundle/atk/AccessibilityUIElementAtk.cpp:

LayoutTests:

Updated expectations to reflect the new mapping to ATK_ROLE_AUDIO and
ATK_ROLE_VIDEO.

* platform/efl/accessibility/media-element-expected.txt: updated
* platform/efl/accessibility/media-emits-object-replacement-expected.txt: updated
* platform/gtk/accessibility/media-controls-panel-title-expected.txt: updated
* platform/gtk/accessibility/media-controls-panel-title.html: updated
* platform/gtk/accessibility/media-element-expected.txt: updated
* platform/gtk/accessibility/media-emits-object-replacement-expected.txt: updated

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

LayoutTests/ChangeLog
LayoutTests/platform/efl/accessibility/media-element-expected.txt
LayoutTests/platform/efl/accessibility/media-emits-object-replacement-expected.txt
LayoutTests/platform/gtk/accessibility/media-controls-panel-title-expected.txt
LayoutTests/platform/gtk/accessibility/media-controls-panel-title.html
LayoutTests/platform/gtk/accessibility/media-element-expected.txt
LayoutTests/platform/gtk/accessibility/media-emits-object-replacement-expected.txt
Source/WebCore/ChangeLog
Source/WebCore/accessibility/atk/WebKitAccessibleWrapperAtk.cpp
Tools/ChangeLog
Tools/WebKitTestRunner/InjectedBundle/atk/AccessibilityUIElementAtk.cpp

index e180b1a..9c86388 100644 (file)
@@ -1,5 +1,22 @@
 2014-11-15  Joanmarie Diggs  <jdiggs@igalia.com>
 
+        AX: [ATK] Video and audio elements should be exposed using ATK's video and audio roles.
+        https://bugs.webkit.org/show_bug.cgi?id=138767
+
+        Reviewed by Chris Fleizach.
+
+        Updated expectations to reflect the new mapping to ATK_ROLE_AUDIO and
+        ATK_ROLE_VIDEO.
+
+        * platform/efl/accessibility/media-element-expected.txt: updated
+        * platform/efl/accessibility/media-emits-object-replacement-expected.txt: updated
+        * platform/gtk/accessibility/media-controls-panel-title-expected.txt: updated
+        * platform/gtk/accessibility/media-controls-panel-title.html: updated
+        * platform/gtk/accessibility/media-element-expected.txt: updated
+        * platform/gtk/accessibility/media-emits-object-replacement-expected.txt: updated
+
+2014-11-15  Joanmarie Diggs  <jdiggs@igalia.com>
+
         AX: [ATK] Expose the footer element as ATK_ROLE_FOOTER
         https://bugs.webkit.org/show_bug.cgi?id=138773
 
index cc4a98b..28a42ca 100644 (file)
@@ -6,7 +6,7 @@ Dump <video> element controller accessibility object tree at 'canplaythrough' ev
 State at 'canplaythrough' event:
 
     description: AXDescription: 
-    role: AXRole: AXEmbedded
+    role: AXRole: AXVideo
 
 
         description: AXDescription: video playback
index cfa128f..a8a193c 100644 (file)
@@ -4,14 +4,14 @@ a  b
 End of test
 AXRole: AXWebArea 
     AXRole: AXSection AXValue: a <obj> b
-        AXRole: AXEmbedded 
+        AXRole: AXVideo 
             AXRole: AXToolbar 
                 AXRole: AXToolbar 
                     AXRole: AXButton 
                     AXRole: AXSlider 
     AXRole: AXGroup AXValue: <\n>
     AXRole: AXSection AXValue: a <obj> b
-        AXRole: AXEmbedded 
+        AXRole: AXAudio 
             AXRole: AXToolbar 
                 AXRole: AXToolbar 
                     AXRole: AXButton 
index 934e0e2..ebf5ef9 100644 (file)
@@ -6,11 +6,11 @@ On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE
 
 PASS mainBlock.role is 'AXRole: AXGroup'
 PASS mainBlock.title is 'AXTitle: '
-PASS element.role is 'AXRole: AXEmbedded'
+PASS element.role is 'AXRole: AXVideo'
 PASS element.title is 'AXTitle: '
 PASS element.role is 'AXRole: AXToolbar'
 PASS element.title is 'AXTitle: Video Playback'
-PASS element.role is 'AXRole: AXEmbedded'
+PASS element.role is 'AXRole: AXAudio'
 PASS element.title is 'AXTitle: '
 PASS element.role is 'AXRole: AXToolbar'
 PASS element.title is 'AXTitle: Audio Playback'
index 7c28301..9364f1f 100644 (file)
@@ -19,7 +19,7 @@ if (window.accessibilityController) {
 
     // Video element.
     var element = mainBlock.childAtIndex(0);
-    shouldBe("element.role", "'AXRole: AXEmbedded'");
+    shouldBe("element.role", "'AXRole: AXVideo'");
     shouldBe("element.title", "'AXTitle: '");
     element = element.childAtIndex(0);
     shouldBe("element.role", "'AXRole: AXToolbar'");
@@ -27,7 +27,7 @@ if (window.accessibilityController) {
 
     // Audio element.
     element = mainBlock.childAtIndex(1);
-    shouldBe("element.role", "'AXRole: AXEmbedded'");
+    shouldBe("element.role", "'AXRole: AXAudio'");
     shouldBe("element.title", "'AXTitle: '");
     element = element.childAtIndex(0);
     shouldBe("element.role", "'AXRole: AXToolbar'");
index 5ecf81b..1f88779 100644 (file)
@@ -6,7 +6,7 @@ Dump <video> element controller accessibility object tree at 'canplaythrough' ev
 State at 'canplaythrough' event:
 
     description: AXDescription: 
-    role: AXRole: AXEmbedded
+    role: AXRole: AXVideo
 
 
         description: AXDescription: Video Playback
index f56ab20..67e1f74 100644 (file)
@@ -4,14 +4,14 @@ a  b
 End of test
 AXRole: AXWebArea 
     AXRole: AXSection AXValue: a <obj> b
-        AXRole: AXEmbedded 
+        AXRole: AXVideo 
             AXRole: AXToolbar 
                 AXRole: AXButton 
                 AXRole: AXSlider 
                 AXRole: AXTimer AXValue: 00:00
     AXRole: AXGroup AXValue: <\n>
     AXRole: AXSection AXValue: a <obj> b
-        AXRole: AXEmbedded 
+        AXRole: AXAudio 
             AXRole: AXToolbar 
                 AXRole: AXButton 
                 AXRole: AXSlider 
index cf9e68a..03d66e0 100644 (file)
@@ -1,5 +1,22 @@
 2014-11-15  Joanmarie Diggs  <jdiggs@igalia.com>
 
+        AX: [ATK] Video and audio elements should be exposed using ATK's video and audio roles.
+        https://bugs.webkit.org/show_bug.cgi?id=138767
+
+        Reviewed by Chris Fleizach.
+
+        Map AudioRole and VideoRole to ATK_ROLE_AUDIO and ATK_ROLE_VIDEO
+        respectively, falling back on the existing mapping to ATK_ROLE_EMBEDDED
+        in environments which have an older version of ATK.
+
+        No new tests are required. Instead, existing expectations were updated
+        to reflect the new mapping.
+
+        * accessibility/atk/WebKitAccessibleWrapperAtk.cpp:
+        (atkRole):
+
+2014-11-15  Joanmarie Diggs  <jdiggs@igalia.com>
+
         AX: [ATK] Expose the footer element as ATK_ROLE_FOOTER
         https://bugs.webkit.org/show_bug.cgi?id=138773
 
index 741d780..546bdf6 100644 (file)
@@ -587,7 +587,13 @@ static AtkRole atkRole(AccessibilityObject* coreObject)
     case UnknownRole:
         return ATK_ROLE_UNKNOWN;
     case AudioRole:
+#if ATK_CHECK_VERSION(2, 11, 3)
+        return ATK_ROLE_AUDIO;
+#endif
     case VideoRole:
+#if ATK_CHECK_VERSION(2, 11, 3)
+        return ATK_ROLE_VIDEO;
+#endif
         return ATK_ROLE_EMBEDDED;
     case ButtonRole:
         return ATK_ROLE_PUSH_BUTTON;
index 5099a6c..67da14e 100644 (file)
@@ -1,3 +1,14 @@
+2014-11-15  Joanmarie Diggs  <jdiggs@igalia.com>
+
+        AX: [ATK] Video and audio elements should be exposed using ATK's video and audio roles.
+        https://bugs.webkit.org/show_bug.cgi?id=138767
+
+        Reviewed by Chris Fleizach.
+
+        Add mappings for ATK_ROLE_AUDIO and ATK_ROLE_VIDEO to roleToString().
+
+        * WebKitTestRunner/InjectedBundle/atk/AccessibilityUIElementAtk.cpp:
+
 2014-11-14  Alexey Proskuryakov  <ap@apple.com>
 
         Add shortcut macros to Mac TestExpectations
index 4dc0706..7a64fa7 100644 (file)
@@ -454,6 +454,8 @@ const gchar* roleToString(AtkObject* object)
 #if ATK_CHECK_VERSION(2, 11, 3)
     case ATK_ROLE_ARTICLE:
         return "AXArticle";
+    case ATK_ROLE_AUDIO:
+        return "AXAudio";
     case ATK_ROLE_DEFINITION:
         return "AXDefinition";
     case ATK_ROLE_LOG:
@@ -464,6 +466,8 @@ const gchar* roleToString(AtkObject* object)
         return "AXMath";
     case ATK_ROLE_TIMER:
         return "AXTimer";
+    case ATK_ROLE_VIDEO:
+        return "AXVideo";
 #endif
 #if ATK_CHECK_VERSION(2, 11, 4)
     case ATK_ROLE_DESCRIPTION_LIST: