Title: [169093] trunk/LayoutTests
Revision
169093
Author
[email protected]
Date
2014-05-19 18:56:41 -0700 (Mon, 19 May 2014)

Log Message

AX: VoiceOver sees the WebArea out of order when topContentInset is used
https://bugs.webkit.org/show_bug.cgi?id=133091

Unreviewed. 
Make the layout test more resilient to platform differences in frame positions.

* platform/mac-wk2/accessibility/content-inset-scrollview-frame-expected.txt:
* platform/mac-wk2/accessibility/content-inset-scrollview-frame.html:

Modified Paths

Diff

Modified: trunk/LayoutTests/ChangeLog (169092 => 169093)


--- trunk/LayoutTests/ChangeLog	2014-05-20 01:53:56 UTC (rev 169092)
+++ trunk/LayoutTests/ChangeLog	2014-05-20 01:56:41 UTC (rev 169093)
@@ -1,3 +1,14 @@
+2014-05-19  Chris Fleizach  <[email protected]>
+
+        AX: VoiceOver sees the WebArea out of order when topContentInset is used
+        https://bugs.webkit.org/show_bug.cgi?id=133091
+
+        Unreviewed. 
+        Make the layout test more resilient to platform differences in frame positions.
+
+        * platform/mac-wk2/accessibility/content-inset-scrollview-frame-expected.txt:
+        * platform/mac-wk2/accessibility/content-inset-scrollview-frame.html:
+
 2014-05-19  Myles C. Maxfield  <[email protected]>
 
         Text decorations do not contribute to visual overflow

Modified: trunk/LayoutTests/platform/mac-wk2/accessibility/content-inset-scrollview-frame-expected.txt (169092 => 169093)


--- trunk/LayoutTests/platform/mac-wk2/accessibility/content-inset-scrollview-frame-expected.txt	2014-05-20 01:53:56 UTC (rev 169092)
+++ trunk/LayoutTests/platform/mac-wk2/accessibility/content-inset-scrollview-frame-expected.txt	2014-05-20 01:56:41 UTC (rev 169093)
@@ -3,11 +3,10 @@
 On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE".
 
 
-Web area role: AXRole: AXWebArea
-WebArea frame: {-10000, 10300}, {800, 600}
-ScrollView role: AXRole: AXScrollArea
-ScrollView frame (0 content inset): {-10000, 10300}, {800, 600}
-ScrollView frame (100 content inset): {-10000, 10200}, {800, 600}
+The position of the web area and the scroll view should be the same when there's no content inset
+PASS webX == scrollViewX && webY == scrollViewY is true
+After setting the content inset, the new y position should be less than the initial
+PASS scrollViewY - scrollView.y is 100
 PASS successfullyParsed is true
 
 TEST COMPLETE

Modified: trunk/LayoutTests/platform/mac-wk2/accessibility/content-inset-scrollview-frame.html (169092 => 169093)


--- trunk/LayoutTests/platform/mac-wk2/accessibility/content-inset-scrollview-frame.html	2014-05-20 01:53:56 UTC (rev 169092)
+++ trunk/LayoutTests/platform/mac-wk2/accessibility/content-inset-scrollview-frame.html	2014-05-20 01:56:41 UTC (rev 169093)
@@ -12,22 +12,22 @@
 
     description("This tests that a page with a content inset will not affect the bounds of the main frame scroll view. It will appear as the same as the web area's sbounds");
 
-    function outputFrame(message, element) {
-        debug(message + ": {" + element.x + ", " + element.y + "}, {" + element.width + ", " + element.height + "}");
-    }
-
     if (window.accessibilityController) {
         var webArea = accessibilityController.rootElement.childAtIndex(0);
-        debug("Web area role: " + webArea.role);
-        outputFrame("WebArea frame", webArea);
+        var webX = webArea.x;
+        var webY = webArea.y;
 
         var scrollView = webArea.parentElement();
-        debug("ScrollView role: " + scrollView.role);
-        outputFrame("ScrollView frame (0 content inset)", scrollView);
-    
+        var scrollViewX = scrollView.x;
+        var scrollViewY = scrollView.y;
+  
+        debug("The position of the web area and the scroll view should be the same when there's no content inset");
+        shouldBeTrue("webX == scrollViewX && webY == scrollViewY");
+ 
         window.internals.setTopContentInset(100);
 
-        outputFrame("ScrollView frame (100 content inset)", scrollView);
+        debug("After setting the content inset, the new y position should be less than the initial");
+        shouldBe("scrollViewY - scrollView.y", "100")
     }
 </script>
 
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to