Title: [137721] trunk/LayoutTests
Revision
137721
Author
[email protected]
Date
2012-12-14 00:13:29 -0800 (Fri, 14 Dec 2012)

Log Message

[GTK] accessibility/ellipsis-text.html is failing
https://bugs.webkit.org/show_bug.cgi?id=98365

Patch by Joanmarie Diggs <[email protected]> on 2012-12-14
Reviewed by Martin Robinson.

The test was failing due to differences in the accessible hierarchies
with respect to which object contains the accessible text. The solution
is to conditionalize the test.

* accessibility/ellipsis-text.html: Modified to handle differences in
the accessible hierarchy
* platform/gtk/TestExpectations: Unskipped the failing test

Modified Paths

Diff

Modified: trunk/LayoutTests/ChangeLog (137720 => 137721)


--- trunk/LayoutTests/ChangeLog	2012-12-14 08:12:22 UTC (rev 137720)
+++ trunk/LayoutTests/ChangeLog	2012-12-14 08:13:29 UTC (rev 137721)
@@ -1,3 +1,18 @@
+2012-12-14  Joanmarie Diggs  <[email protected]>
+
+        [GTK] accessibility/ellipsis-text.html is failing
+        https://bugs.webkit.org/show_bug.cgi?id=98365
+
+        Reviewed by Martin Robinson.
+
+        The test was failing due to differences in the accessible hierarchies
+        with respect to which object contains the accessible text. The solution
+        is to conditionalize the test.
+
+        * accessibility/ellipsis-text.html: Modified to handle differences in
+        the accessible hierarchy
+        * platform/gtk/TestExpectations: Unskipped the failing test
+
 2012-12-13  Yoshifumi Inoue  <[email protected]>
 
         Copy platform/win/fast/events/panScroll-* to platform/chromium-win/fast/events

Modified: trunk/LayoutTests/accessibility/ellipsis-text.html (137720 => 137721)


--- trunk/LayoutTests/accessibility/ellipsis-text.html	2012-12-14 08:12:22 UTC (rev 137720)
+++ trunk/LayoutTests/accessibility/ellipsis-text.html	2012-12-14 08:13:29 UTC (rev 137721)
@@ -20,13 +20,13 @@
             // The width of the ellipsis text should be short.
             document.getElementById("text-ellipsis").focus();
             var textContainer = accessibilityController.focusedElement;
-            var textNode = textContainer.childAtIndex(0);
+            var textNode = textContainer.childrenCount ? textContainer.childAtIndex(0) : textContainer;
             var clippedWidth = textNode.width;
 
             // The width of non-ellipsis'd text should be longer.
             document.getElementById("text-noellipsis").focus();
             textContainer = accessibilityController.focusedElement;
-            textNode = textContainer.childAtIndex(0);
+            textNode = textContainer.childrenCount ? textContainer.childAtIndex(0) : textContainer;
             var fullWidth = textNode.width;
 
             shouldBeTrue("clippedWidth < fullWidth");

Modified: trunk/LayoutTests/platform/gtk/TestExpectations (137720 => 137721)


--- trunk/LayoutTests/platform/gtk/TestExpectations	2012-12-14 08:12:22 UTC (rev 137720)
+++ trunk/LayoutTests/platform/gtk/TestExpectations	2012-12-14 08:13:29 UTC (rev 137721)
@@ -769,7 +769,6 @@
 webkit.org/b/98360 accessibility/aria-used-on-image-maps.html [ Failure ]
 webkit.org/b/98361 accessibility/button-press-action.html [ Failure ]
 webkit.org/b/98363 accessibility/canvas-fallback-content-2.html [ Failure ]
-webkit.org/b/98365 accessibility/ellipsis-text.html [ Failure ]
 webkit.org/b/98370 accessibility/loading-iframe-sends-notification.html [ Failure ]
 webkit.org/b/98371 accessibility/loading-iframe-updates-axtree.html [ Failure ]
 webkit.org/b/98372 accessibility/onclick-handlers.html [ Failure ]
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to