Title: [107230] trunk/LayoutTests
Revision
107230
Author
[email protected]
Date
2012-02-09 05:58:49 -0800 (Thu, 09 Feb 2012)

Log Message

2012-02-09  Nikolas Zimmermann  <[email protected]>

        Not reviewed. Make the hit-test-with-br.xhtml work on non-Mac platforms.
        it was skipped on Chromium Win/Linux before, as the click on the <rect> never suceeeded.
        On  Mac the coordinate 10x70, was the upper left corner of the rectangle, so if the font
        size is just slightly larger, or the size of the <br/>, then we failed to fire the click
        event - so just click at 50x80, to be on the safe side.

        * svg/custom/hit-test-with-br.xhtml:

Modified Paths

Diff

Modified: trunk/LayoutTests/ChangeLog (107229 => 107230)


--- trunk/LayoutTests/ChangeLog	2012-02-09 13:53:18 UTC (rev 107229)
+++ trunk/LayoutTests/ChangeLog	2012-02-09 13:58:49 UTC (rev 107230)
@@ -1,5 +1,15 @@
 2012-02-09  Nikolas Zimmermann  <[email protected]>
 
+        Not reviewed. Make the hit-test-with-br.xhtml work on non-Mac platforms.
+        it was skipped on Chromium Win/Linux before, as the click on the <rect> never suceeeded.
+        On  Mac the coordinate 10x70, was the upper left corner of the rectangle, so if the font
+        size is just slightly larger, or the size of the <br/>, then we failed to fire the click
+        event - so just click at 50x80, to be on the safe side.
+
+        * svg/custom/hit-test-with-br.xhtml:
+
+2012-02-09  Nikolas Zimmermann  <[email protected]>
+
         [Chromium] Layout Test svg/as-object/nested-embedded-svg-size-changes.html times out
         https://bugs.webkit.org/show_bug.cgi?id=77183
 

Modified: trunk/LayoutTests/svg/custom/hit-test-with-br.xhtml (107229 => 107230)


--- trunk/LayoutTests/svg/custom/hit-test-with-br.xhtml	2012-02-09 13:53:18 UTC (rev 107229)
+++ trunk/LayoutTests/svg/custom/hit-test-with-br.xhtml	2012-02-09 13:58:49 UTC (rev 107230)
@@ -12,13 +12,9 @@
 function repaintTest() {
     if (window.eventSender) {
         layoutTestController.waitUntilDone();
-
-        // This timeout should be unncessary, but as the test currently times out on Chromium, I'm trying with this timeout.
-        setTimeout(function() {
-            eventSender.mouseMoveTo(10, 70);
-            eventSender.mouseDown();
-            eventSender.mouseUp();
-        }, 0);
+        eventSender.mouseMoveTo(50, 80);
+        eventSender.mouseDown();
+        eventSender.mouseUp();
     }
 }
 </script>
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to