Title: [173556] trunk/LayoutTests
Revision
173556
Author
[email protected]
Date
2014-09-11 22:32:33 -0700 (Thu, 11 Sep 2014)

Log Message

fast/forms/submit-to-blank-multiple-times.html crashes with JSC_slowPathAllocsBetweenGCs
https://bugs.webkit.org/show_bug.cgi?id=136716

Reviewed by Darin Adler.

This test used a fake timeout handler, which is not needed, because WRT/WKTR do it for you.
Any time the test took over 2 seconds, we would dump and disable waiting for notifyDone,
but the test would still try to open windows after that.

* fast/forms/submit-to-blank-multiple-times.html:
* platform/mac/TestExpectations:

Modified Paths

Diff

Modified: trunk/LayoutTests/ChangeLog (173555 => 173556)


--- trunk/LayoutTests/ChangeLog	2014-09-12 05:08:39 UTC (rev 173555)
+++ trunk/LayoutTests/ChangeLog	2014-09-12 05:32:33 UTC (rev 173556)
@@ -1,3 +1,17 @@
+2014-09-11  Alexey Proskuryakov  <[email protected]>
+
+        fast/forms/submit-to-blank-multiple-times.html crashes with JSC_slowPathAllocsBetweenGCs
+        https://bugs.webkit.org/show_bug.cgi?id=136716
+
+        Reviewed by Darin Adler.
+
+        This test used a fake timeout handler, which is not needed, because WRT/WKTR do it for you.
+        Any time the test took over 2 seconds, we would dump and disable waiting for notifyDone,
+        but the test would still try to open windows after that.
+
+        * fast/forms/submit-to-blank-multiple-times.html:
+        * platform/mac/TestExpectations:
+
 2014-09-11  Shivakumar JM  <[email protected]>
 
         The overrideMimeType in XMLHttpRequest should throw the exception.

Modified: trunk/LayoutTests/fast/forms/submit-to-blank-multiple-times.html (173555 => 173556)


--- trunk/LayoutTests/fast/forms/submit-to-blank-multiple-times.html	2014-09-12 05:08:39 UTC (rev 173555)
+++ trunk/LayoutTests/fast/forms/submit-to-blank-multiple-times.html	2014-09-12 05:32:33 UTC (rev 173556)
@@ -20,6 +20,7 @@
         if (window.testRunner) {
             testRunner.dumpAsText();
             testRunner.setCanOpenWindows();
+            testRunner.setCloseRemainingWindowsWhenComplete();
             testRunner.waitUntilDone();
         }
         var numberTimesToClick = 2;
@@ -29,8 +30,6 @@
                 return;
             
             click();
-            // Set a timeout to notify done in case the test fails.
-            setTimeout(notifyDone, 2000);
         };
         function log(message) {
             var span = document.createElement("span");
@@ -65,7 +64,6 @@
             eventSender.keyDown(' ');
         }
         function notifyDone() {
-            testRunner.setCloseRemainingWindowsWhenComplete();
             testRunner.notifyDone();
         }
         </script>

Modified: trunk/LayoutTests/platform/mac/TestExpectations (173555 => 173556)


--- trunk/LayoutTests/platform/mac/TestExpectations	2014-09-12 05:08:39 UTC (rev 173555)
+++ trunk/LayoutTests/platform/mac/TestExpectations	2014-09-12 05:32:33 UTC (rev 173556)
@@ -1154,8 +1154,6 @@
 
 webkit.org/b/135133 [ MountainLion ] fast/layers/no-clipping-overflow-hidden-added-after-transform.html [ Pass ImageOnlyFailure ]
 
-webkit.org/b/136716 fast/forms/submit-to-blank-multiple-times.html [ Skip ]
-
 webkit.org/b/136717 js/dom/string-replace-exception-crash.html [ Skip ]
 
 webkit.org/b/136718 fast/dynamic/layer-no-longer-paginated.html [ Skip ]
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to