Title: [123886] trunk/LayoutTests
Revision
123886
Author
[email protected]
Date
2012-07-27 10:48:46 -0700 (Fri, 27 Jul 2012)

Log Message

Lose a timer in prerender-remove-after-stop.html
https://bugs.webkit.org/show_bug.cgi?id=92425

Patch by Gavin Peters <[email protected]> on 2012-07-27
Reviewed by Nate Chapin.

japhet suggested that the timer wasn't needed, and testing revealed that the test works without it. I've tested that if the bug is reintroduced
that it crashes, and of course it doesn't crash with the fix. So bye-bye setTimeout!

* fast/dom/HTMLLinkElement/prerender-remove-after-stop.html:

Modified Paths

Diff

Modified: trunk/LayoutTests/ChangeLog (123885 => 123886)


--- trunk/LayoutTests/ChangeLog	2012-07-27 17:39:54 UTC (rev 123885)
+++ trunk/LayoutTests/ChangeLog	2012-07-27 17:48:46 UTC (rev 123886)
@@ -1,3 +1,15 @@
+2012-07-27  Gavin Peters  <[email protected]>
+
+        Lose a timer in prerender-remove-after-stop.html
+        https://bugs.webkit.org/show_bug.cgi?id=92425
+
+        Reviewed by Nate Chapin.
+
+        japhet suggested that the timer wasn't needed, and testing revealed that the test works without it. I've tested that if the bug is reintroduced
+        that it crashes, and of course it doesn't crash with the fix. So bye-bye setTimeout!
+
+        * fast/dom/HTMLLinkElement/prerender-remove-after-stop.html:
+
 2012-07-27  Balazs Ankes  <[email protected]>
 
         [Qt][WK2] Gardening after r123786. It made 5 fast/animation tests fail

Modified: trunk/LayoutTests/fast/dom/HTMLLinkElement/prerender-remove-after-stop.html (123885 => 123886)


--- trunk/LayoutTests/fast/dom/HTMLLinkElement/prerender-remove-after-stop.html	2012-07-27 17:39:54 UTC (rev 123885)
+++ trunk/LayoutTests/fast/dom/HTMLLinkElement/prerender-remove-after-stop.html	2012-07-27 17:48:46 UTC (rev 123886)
@@ -12,7 +12,7 @@
     return function() {
         var element = doc.getElementById(id);
         element.parentNode.removeChild(element);
-        setTimeout(done, 0);
+        done();
     }
 }
 
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to