Title: [160323] trunk
Revision
160323
Author
mario.pr...@samsung.com
Date
2013-12-09 12:16:50 -0800 (Mon, 09 Dec 2013)

Log Message

[ATK] Translate ATK_ROLE_SECTION into "AXSection" in DRT/WKTR
https://bugs.webkit.org/show_bug.cgi?id=125456

Reviewed by Chris Fleizach.

Tools:

Return 'AXSection' for section roles instead of 'AXDiv'.

* DumpRenderTree/atk/AccessibilityUIElementAtk.cpp:
* WebKitTestRunner/InjectedBundle/atk/AccessibilityUIElementAtk.cpp:

LayoutTests:

Update expectations for GTK and EFL that were expecting AXDiv for
section roles, so they now expect AXSection.

* accessibility/adjacent-continuations-cause-assertion-failure-expected.txt:
* accessibility/div-within-anchors-causes-crash-expected.txt:
* platform/efl-wk1/accessibility/image-map2-expected.txt:
* platform/efl-wk1/accessibility/transformed-element-expected.txt:
* platform/efl-wk2/accessibility/image-map2-expected.txt:
* platform/efl-wk2/accessibility/transformed-element-expected.txt:
* platform/efl/accessibility/media-emits-object-replacement-expected.txt:
* platform/gtk/accessibility/aria-roles-unignored-expected.txt:
* platform/gtk/accessibility/aria-roles-unignored.html:
* platform/gtk/accessibility/entry-and-password-expected.txt:
* platform/gtk/accessibility/image-map2-expected.txt:
* platform/gtk/accessibility/media-emits-object-replacement-expected.txt:
* platform/gtk/accessibility/object-with-title-expected.txt:
* platform/gtk/accessibility/object-with-title.html:
* platform/gtk/accessibility/replaced-objects-in-anonymous-blocks-expected.txt:
* platform/gtk/accessibility/spans-paragraphs-and-divs-expected.txt:
* platform/gtk/accessibility/spans-paragraphs-and-divs.html:
* platform/gtk/accessibility/transformed-element-expected.txt:

Modified Paths

Diff

Modified: trunk/LayoutTests/ChangeLog (160322 => 160323)


--- trunk/LayoutTests/ChangeLog	2013-12-09 19:45:21 UTC (rev 160322)
+++ trunk/LayoutTests/ChangeLog	2013-12-09 20:16:50 UTC (rev 160323)
@@ -1,3 +1,32 @@
+2013-12-09  Mario Sanchez Prada  <mario.pr...@samsung.com>
+
+        [ATK] Translate ATK_ROLE_SECTION into "AXSection" in DRT/WKTR
+        https://bugs.webkit.org/show_bug.cgi?id=125456
+
+        Reviewed by Chris Fleizach.
+
+        Update expectations for GTK and EFL that were expecting AXDiv for
+        section roles, so they now expect AXSection.
+
+        * accessibility/adjacent-continuations-cause-assertion-failure-expected.txt:
+        * accessibility/div-within-anchors-causes-crash-expected.txt:
+        * platform/efl-wk1/accessibility/image-map2-expected.txt:
+        * platform/efl-wk1/accessibility/transformed-element-expected.txt:
+        * platform/efl-wk2/accessibility/image-map2-expected.txt:
+        * platform/efl-wk2/accessibility/transformed-element-expected.txt:
+        * platform/efl/accessibility/media-emits-object-replacement-expected.txt:
+        * platform/gtk/accessibility/aria-roles-unignored-expected.txt:
+        * platform/gtk/accessibility/aria-roles-unignored.html:
+        * platform/gtk/accessibility/entry-and-password-expected.txt:
+        * platform/gtk/accessibility/image-map2-expected.txt:
+        * platform/gtk/accessibility/media-emits-object-replacement-expected.txt:
+        * platform/gtk/accessibility/object-with-title-expected.txt:
+        * platform/gtk/accessibility/object-with-title.html:
+        * platform/gtk/accessibility/replaced-objects-in-anonymous-blocks-expected.txt:
+        * platform/gtk/accessibility/spans-paragraphs-and-divs-expected.txt:
+        * platform/gtk/accessibility/spans-paragraphs-and-divs.html:
+        * platform/gtk/accessibility/transformed-element-expected.txt:
+
 2013-12-09  Peter Molnar  <pmolnar.u-sze...@partner.samsung.com>
 
         Fix handling of 'inherit' and 'initial' for grid lines.

Modified: trunk/LayoutTests/accessibility/adjacent-continuations-cause-assertion-failure-expected.txt (160322 => 160323)


--- trunk/LayoutTests/accessibility/adjacent-continuations-cause-assertion-failure-expected.txt	2013-12-09 19:45:21 UTC (rev 160322)
+++ trunk/LayoutTests/accessibility/adjacent-continuations-cause-assertion-failure-expected.txt	2013-12-09 20:16:50 UTC (rev 160323)
@@ -8,8 +8,8 @@
 
 
 AXRole: AXWebArea 
-    AXRole: AXDiv AXValue: y
-    AXRole: AXDiv AXValue: End of test
+    AXRole: AXSection AXValue: y
+    AXRole: AXSection AXValue: End of test
 PASS successfullyParsed is true
 
 TEST COMPLETE

Modified: trunk/LayoutTests/accessibility/div-within-anchors-causes-crash-expected.txt (160322 => 160323)


--- trunk/LayoutTests/accessibility/div-within-anchors-causes-crash-expected.txt	2013-12-09 19:45:21 UTC (rev 160322)
+++ trunk/LayoutTests/accessibility/div-within-anchors-causes-crash-expected.txt	2013-12-09 20:16:50 UTC (rev 160323)
@@ -2,10 +2,10 @@
 Before:
 AXRole: AXWebArea 
     AXRole: AXLink AXValue: 
-    AXRole: AXDiv AXValue: End of test
+    AXRole: AXSection AXValue: End of test
 After:
 AXRole: AXWebArea 
-    AXRole: AXDiv AXValue: End of test
+    AXRole: AXSection AXValue: End of test
 This can cause a crash.
 
 On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE".

Modified: trunk/LayoutTests/platform/efl/accessibility/media-emits-object-replacement-expected.txt (160322 => 160323)


--- trunk/LayoutTests/platform/efl/accessibility/media-emits-object-replacement-expected.txt	2013-12-09 19:45:21 UTC (rev 160322)
+++ trunk/LayoutTests/platform/efl/accessibility/media-emits-object-replacement-expected.txt	2013-12-09 20:16:50 UTC (rev 160323)
@@ -3,20 +3,20 @@
 a  b
 End of test
 AXRole: AXWebArea 
-    AXRole: AXDiv AXValue: a <obj> b
+    AXRole: AXSection AXValue: a <obj> b
         AXRole: AXEmbedded 
             AXRole: AXToolbar 
                 AXRole: AXToolbar 
                     AXRole: AXButton 
                     AXRole: AXSlider 
     AXRole: AXGroup AXValue: <\n>
-    AXRole: AXDiv AXValue: a <obj> b
+    AXRole: AXSection AXValue: a <obj> b
         AXRole: AXEmbedded 
             AXRole: AXToolbar 
                 AXRole: AXToolbar 
                     AXRole: AXButton 
                     AXRole: AXSlider 
-    AXRole: AXDiv AXValue: End of test
+    AXRole: AXSection AXValue: End of test
 This tests ensures that if video or audio tags are used, they will emit an object replacement character in a range for string operation.
 
 On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE".

Modified: trunk/LayoutTests/platform/efl-wk1/accessibility/image-map2-expected.txt (160322 => 160323)


--- trunk/LayoutTests/platform/efl-wk1/accessibility/image-map2-expected.txt	2013-12-09 19:45:21 UTC (rev 160322)
+++ trunk/LayoutTests/platform/efl-wk1/accessibility/image-map2-expected.txt	2013-12-09 20:16:50 UTC (rev 160323)
@@ -1,6 +1,6 @@
 Image map - test 2 - 2 Links (alt tags)
 ----------------------
-AXRole: AXDiv
+AXRole: AXSection
 AXParent: AXWebArea
 AXChildren: 0
 AXPosition: { 8.000000, 8.000000 }

Modified: trunk/LayoutTests/platform/efl-wk1/accessibility/transformed-element-expected.txt (160322 => 160323)


--- trunk/LayoutTests/platform/efl-wk1/accessibility/transformed-element-expected.txt	2013-12-09 19:45:21 UTC (rev 160322)
+++ trunk/LayoutTests/platform/efl-wk1/accessibility/transformed-element-expected.txt	2013-12-09 20:16:50 UTC (rev 160323)
@@ -1,5 +1,5 @@
 hello
-AXRole: AXDiv
+AXRole: AXSection
 AXParent: AXWebArea
 AXChildren: 1
 AXPosition: { 8.000000, 9.000000 }

Modified: trunk/LayoutTests/platform/efl-wk2/accessibility/image-map2-expected.txt (160322 => 160323)


--- trunk/LayoutTests/platform/efl-wk2/accessibility/image-map2-expected.txt	2013-12-09 19:45:21 UTC (rev 160322)
+++ trunk/LayoutTests/platform/efl-wk2/accessibility/image-map2-expected.txt	2013-12-09 20:16:50 UTC (rev 160323)
@@ -1,6 +1,6 @@
 Image map - test 2 - 2 Links (alt tags)
 ----------------------
-AXRole: AXDiv
+AXRole: AXSection
 AXParent: AXWebArea
 AXChildren: 0
 AXPosition: { 0.000000, 0.000000 }

Modified: trunk/LayoutTests/platform/efl-wk2/accessibility/transformed-element-expected.txt (160322 => 160323)


--- trunk/LayoutTests/platform/efl-wk2/accessibility/transformed-element-expected.txt	2013-12-09 19:45:21 UTC (rev 160322)
+++ trunk/LayoutTests/platform/efl-wk2/accessibility/transformed-element-expected.txt	2013-12-09 20:16:50 UTC (rev 160323)
@@ -1,5 +1,5 @@
 hello
-AXRole: AXDiv
+AXRole: AXSection
 AXParent: AXWebArea
 AXChildren: 1
 AXPosition: { 0.000000, 0.000000 }

Modified: trunk/LayoutTests/platform/gtk/accessibility/aria-roles-unignored-expected.txt (160322 => 160323)


--- trunk/LayoutTests/platform/gtk/accessibility/aria-roles-unignored-expected.txt	2013-12-09 19:45:21 UTC (rev 160322)
+++ trunk/LayoutTests/platform/gtk/accessibility/aria-roles-unignored-expected.txt	2013-12-09 20:16:50 UTC (rev 160323)
@@ -18,7 +18,7 @@
 PASS element.role is 'AXRole: AXHeading'
 PASS element.role is 'AXRole: AXForm'
 PASS element.role is 'AXRole: AXButton'
-PASS element.role is 'AXRole: AXDiv'
+PASS element.role is 'AXRole: AXSection'
 PASS element.role is 'AXRole: AXTextField'
 PASS successfullyParsed is true
 

Modified: trunk/LayoutTests/platform/gtk/accessibility/aria-roles-unignored.html (160322 => 160323)


--- trunk/LayoutTests/platform/gtk/accessibility/aria-roles-unignored.html	2013-12-09 19:45:21 UTC (rev 160322)
+++ trunk/LayoutTests/platform/gtk/accessibility/aria-roles-unignored.html	2013-12-09 20:16:50 UTC (rev 160323)
@@ -51,7 +51,7 @@
 
     // Divs
     element = webArea.childAtIndex(5);
-    shouldBe("element.role", "'AXRole: AXDiv'");
+    shouldBe("element.role", "'AXRole: AXSection'");
     element = webArea.childAtIndex(6);
     shouldBe("element.role", "'AXRole: AXTextField'");
 }

Modified: trunk/LayoutTests/platform/gtk/accessibility/entry-and-password-expected.txt (160322 => 160323)


--- trunk/LayoutTests/platform/gtk/accessibility/entry-and-password-expected.txt	2013-12-09 19:45:21 UTC (rev 160322)
+++ trunk/LayoutTests/platform/gtk/accessibility/entry-and-password-expected.txt	2013-12-09 20:16:50 UTC (rev 160323)
@@ -4,7 +4,7 @@
     AXRole: AXForm AXValue: <obj> <obj>
         AXRole: AXTextField AXValue: 123
         AXRole: AXPasswordField AXValue: •••
-    AXRole: AXDiv AXValue: End of test
+    AXRole: AXSection AXValue: End of test
 This tests that the text associated with entry and password fields is correct.
 
 On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE".

Modified: trunk/LayoutTests/platform/gtk/accessibility/image-map2-expected.txt (160322 => 160323)


--- trunk/LayoutTests/platform/gtk/accessibility/image-map2-expected.txt	2013-12-09 19:45:21 UTC (rev 160322)
+++ trunk/LayoutTests/platform/gtk/accessibility/image-map2-expected.txt	2013-12-09 20:16:50 UTC (rev 160323)
@@ -1,6 +1,6 @@
 Image map - test 2 - 2 Links (alt tags)
 ----------------------
-AXRole: AXDiv
+AXRole: AXSection
 AXParent: AXWebArea
 AXChildren: 0
 AXPosition: { 8.000000, 8.000000 }

Modified: trunk/LayoutTests/platform/gtk/accessibility/media-emits-object-replacement-expected.txt (160322 => 160323)


--- trunk/LayoutTests/platform/gtk/accessibility/media-emits-object-replacement-expected.txt	2013-12-09 19:45:21 UTC (rev 160322)
+++ trunk/LayoutTests/platform/gtk/accessibility/media-emits-object-replacement-expected.txt	2013-12-09 20:16:50 UTC (rev 160323)
@@ -3,20 +3,20 @@
 a  b
 End of test
 AXRole: AXWebArea 
-    AXRole: AXDiv AXValue: a <obj> b
+    AXRole: AXSection AXValue: a <obj> b
         AXRole: AXEmbedded 
             AXRole: AXToolbar 
                 AXRole: AXToolbar 
                     AXRole: AXButton 
                     AXRole: AXSlider 
     AXRole: AXGroup AXValue: <\n>
-    AXRole: AXDiv AXValue: a <obj> b
+    AXRole: AXSection AXValue: a <obj> b
         AXRole: AXEmbedded 
             AXRole: AXToolbar 
                 AXRole: AXToolbar 
                     AXRole: AXButton 
                     AXRole: AXSlider 
-    AXRole: AXDiv AXValue: End of test
+    AXRole: AXSection AXValue: End of test
 This tests ensures that if video or audio tags are used, they will emit an object replacement character in a range for string operation.
 
 On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE".

Modified: trunk/LayoutTests/platform/gtk/accessibility/object-with-title-expected.txt (160322 => 160323)


--- trunk/LayoutTests/platform/gtk/accessibility/object-with-title-expected.txt	2013-12-09 19:45:21 UTC (rev 160322)
+++ trunk/LayoutTests/platform/gtk/accessibility/object-with-title-expected.txt	2013-12-09 20:16:50 UTC (rev 160323)
@@ -8,7 +8,7 @@
 
 
 PASS child.role is 'AXRole: AXList'
-PASS child.role is 'AXRole: AXDiv'
+PASS child.role is 'AXRole: AXSection'
 PASS child.role is 'AXRole: AXList'
 PASS successfullyParsed is true
 

Modified: trunk/LayoutTests/platform/gtk/accessibility/object-with-title.html (160322 => 160323)


--- trunk/LayoutTests/platform/gtk/accessibility/object-with-title.html	2013-12-09 19:45:21 UTC (rev 160322)
+++ trunk/LayoutTests/platform/gtk/accessibility/object-with-title.html	2013-12-09 20:16:50 UTC (rev 160323)
@@ -23,7 +23,7 @@
         shouldBe("child.role", "'AXRole: AXList'");
 
         child = webArea.childAtIndex(1);
-        shouldBe("child.role", "'AXRole: AXDiv'");
+        shouldBe("child.role", "'AXRole: AXSection'");
 
         child = child.childAtIndex(0);
         shouldBe("child.role", "'AXRole: AXList'");

Modified: trunk/LayoutTests/platform/gtk/accessibility/replaced-objects-in-anonymous-blocks-expected.txt (160322 => 160323)


--- trunk/LayoutTests/platform/gtk/accessibility/replaced-objects-in-anonymous-blocks-expected.txt	2013-12-09 19:45:21 UTC (rev 160322)
+++ trunk/LayoutTests/platform/gtk/accessibility/replaced-objects-in-anonymous-blocks-expected.txt	2013-12-09 20:16:50 UTC (rev 160323)
@@ -20,12 +20,12 @@
                 AXRole: AXWebArea 
     AXRole: AXGroup AXValue: <obj>
         AXRole: AXTextField AXValue: text area
-    AXRole: AXDiv AXValue: Div<\n><obj>
+    AXRole: AXSection AXValue: Div<\n><obj>
         AXRole: AXButton 
     AXRole: AXHeading AXValue: Heading<\n><obj><\n><obj>
         AXRole: AXButton 
         AXRole: AXImage 
-    AXRole: AXDiv AXValue: End of test
+    AXRole: AXSection AXValue: End of test
 This tests that the text associated with replaced objects in an anonymous block is correct.
 
 On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE".

Modified: trunk/LayoutTests/platform/gtk/accessibility/spans-paragraphs-and-divs-expected.txt (160322 => 160323)


--- trunk/LayoutTests/platform/gtk/accessibility/spans-paragraphs-and-divs-expected.txt	2013-12-09 19:45:21 UTC (rev 160322)
+++ trunk/LayoutTests/platform/gtk/accessibility/spans-paragraphs-and-divs-expected.txt	2013-12-09 20:16:50 UTC (rev 160323)
@@ -34,17 +34,17 @@
 PASS link.childrenCount is 0
 PASS element.role is 'AXRole: AXParagraph'
 PASS element.childrenCount is 0
-PASS element.role is 'AXRole: AXDiv'
+PASS element.role is 'AXRole: AXSection'
 PASS element.childrenCount is 1
 PASS link.role is 'AXRole: AXLink'
 PASS link.childrenCount is 0
-PASS element.role is 'AXRole: AXDiv'
+PASS element.role is 'AXRole: AXSection'
 PASS element.childrenCount is 0
-PASS element.role is 'AXRole: AXDiv'
+PASS element.role is 'AXRole: AXSection'
 PASS element.childrenCount is 1
 PASS link.role is 'AXRole: AXLink'
 PASS link.childrenCount is 0
-PASS element.role is 'AXRole: AXDiv'
+PASS element.role is 'AXRole: AXSection'
 PASS element.childrenCount is 0
 PASS successfullyParsed is true
 

Modified: trunk/LayoutTests/platform/gtk/accessibility/spans-paragraphs-and-divs.html (160322 => 160323)


--- trunk/LayoutTests/platform/gtk/accessibility/spans-paragraphs-and-divs.html	2013-12-09 19:45:21 UTC (rev 160322)
+++ trunk/LayoutTests/platform/gtk/accessibility/spans-paragraphs-and-divs.html	2013-12-09 20:16:50 UTC (rev 160323)
@@ -83,7 +83,7 @@
     shouldBe("element.childrenCount", "0");
 
     element = webArea.childAtIndex(4);
-    shouldBe("element.role", "'AXRole: AXDiv'");
+    shouldBe("element.role", "'AXRole: AXSection'");
     shouldBe("element.childrenCount", "1");
 
     link = element.childAtIndex(0);
@@ -91,11 +91,11 @@
     shouldBe("link.childrenCount", "0");
 
     element = webArea.childAtIndex(5);
-    shouldBe("element.role", "'AXRole: AXDiv'");
+    shouldBe("element.role", "'AXRole: AXSection'");
     shouldBe("element.childrenCount", "0");
 
     element = webArea.childAtIndex(6);
-    shouldBe("element.role", "'AXRole: AXDiv'");
+    shouldBe("element.role", "'AXRole: AXSection'");
     shouldBe("element.childrenCount", "1");
 
     link = element.childAtIndex(0);
@@ -103,7 +103,7 @@
     shouldBe("link.childrenCount", "0");
 
     element = webArea.childAtIndex(7);
-    shouldBe("element.role", "'AXRole: AXDiv'");
+    shouldBe("element.role", "'AXRole: AXSection'");
     shouldBe("element.childrenCount", "0");
 }
 

Modified: trunk/LayoutTests/platform/gtk/accessibility/transformed-element-expected.txt (160322 => 160323)


--- trunk/LayoutTests/platform/gtk/accessibility/transformed-element-expected.txt	2013-12-09 19:45:21 UTC (rev 160322)
+++ trunk/LayoutTests/platform/gtk/accessibility/transformed-element-expected.txt	2013-12-09 20:16:50 UTC (rev 160323)
@@ -1,5 +1,5 @@
 hello
-AXRole: AXDiv
+AXRole: AXSection
 AXParent: AXWebArea
 AXChildren: 1
 AXPosition: { 8.000000, 9.000000 }

Modified: trunk/Tools/ChangeLog (160322 => 160323)


--- trunk/Tools/ChangeLog	2013-12-09 19:45:21 UTC (rev 160322)
+++ trunk/Tools/ChangeLog	2013-12-09 20:16:50 UTC (rev 160323)
@@ -1,3 +1,15 @@
+2013-12-09  Mario Sanchez Prada  <mario.pr...@samsung.com>
+
+        [ATK] Translate ATK_ROLE_SECTION into "AXSection" in DRT/WKTR
+        https://bugs.webkit.org/show_bug.cgi?id=125456
+
+        Reviewed by Chris Fleizach.
+
+        Return 'AXSection' for section roles instead of 'AXDiv'.
+
+        * DumpRenderTree/atk/AccessibilityUIElementAtk.cpp:
+        * WebKitTestRunner/InjectedBundle/atk/AccessibilityUIElementAtk.cpp:
+
 2013-12-09  Brian J. Burg  <b...@cs.washington.edu>
 
         check-webkit-style should check for extra newlines at EOF

Modified: trunk/Tools/DumpRenderTree/atk/AccessibilityUIElementAtk.cpp (160322 => 160323)


--- trunk/Tools/DumpRenderTree/atk/AccessibilityUIElementAtk.cpp	2013-12-09 19:45:21 UTC (rev 160322)
+++ trunk/Tools/DumpRenderTree/atk/AccessibilityUIElementAtk.cpp	2013-12-09 20:16:50 UTC (rev 160323)
@@ -258,7 +258,7 @@
     case ATK_ROLE_SCROLL_PANE:
         return "AXScrollArea";
     case ATK_ROLE_SECTION:
-        return "AXDiv";
+        return "AXSection";
     case ATK_ROLE_SEPARATOR:
         return "AXHorizontalRule";
     case ATK_ROLE_SLIDER:

Modified: trunk/Tools/WebKitTestRunner/InjectedBundle/atk/AccessibilityUIElementAtk.cpp (160322 => 160323)


--- trunk/Tools/WebKitTestRunner/InjectedBundle/atk/AccessibilityUIElementAtk.cpp	2013-12-09 19:45:21 UTC (rev 160322)
+++ trunk/Tools/WebKitTestRunner/InjectedBundle/atk/AccessibilityUIElementAtk.cpp	2013-12-09 20:16:50 UTC (rev 160323)
@@ -347,7 +347,7 @@
     case ATK_ROLE_SCROLL_PANE:
         return "AXScrollArea";
     case ATK_ROLE_SECTION:
-        return "AXDiv";
+        return "AXSection";
     case ATK_ROLE_SEPARATOR:
         return "AXHorizontalRule";
     case ATK_ROLE_SLIDER:
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to