Title: [155472] trunk/LayoutTests
Revision
155472
Author
[email protected]
Date
2013-09-10 14:15:29 -0700 (Tue, 10 Sep 2013)

Log Message

Fix race in fast/loader/onunload-form-submit-crash.html
https://bugs.webkit.org/show_bug.cgi?id=121122

Reviewed by Alexey Proskuryakov.

Merge https://chromium.googlesource.com/chromium/blink/+/c7ab0900c147f9a2f4c69d62b6460f98abc5d796

The setTimeout was sometimes firing before the form was parsed, so document.myForm would be null.

* fast/loader/onunload-form-submit-crash.html:

Modified Paths

Diff

Modified: trunk/LayoutTests/ChangeLog (155471 => 155472)


--- trunk/LayoutTests/ChangeLog	2013-09-10 21:00:42 UTC (rev 155471)
+++ trunk/LayoutTests/ChangeLog	2013-09-10 21:15:29 UTC (rev 155472)
@@ -1,3 +1,16 @@
+2013-09-10  Ryosuke Niwa  <[email protected]>
+
+        Fix race in fast/loader/onunload-form-submit-crash.html
+        https://bugs.webkit.org/show_bug.cgi?id=121122
+
+        Reviewed by Alexey Proskuryakov.
+
+        Merge https://chromium.googlesource.com/chromium/blink/+/c7ab0900c147f9a2f4c69d62b6460f98abc5d796
+
+        The setTimeout was sometimes firing before the form was parsed, so document.myForm would be null.
+
+        * fast/loader/onunload-form-submit-crash.html:
+
 2013-09-10  Joseph Pecoraro  <[email protected]>
 
         Web Inspector: [JSC] Caught exception is treated as uncaught

Modified: trunk/LayoutTests/fast/loader/onunload-form-submit-crash.html (155471 => 155472)


--- trunk/LayoutTests/fast/loader/onunload-form-submit-crash.html	2013-09-10 21:00:42 UTC (rev 155471)
+++ trunk/LayoutTests/fast/loader/onunload-form-submit-crash.html	2013-09-10 21:15:29 UTC (rev 155472)
@@ -1,5 +1,7 @@
 <html>
     <body _onUnload_="document.myForm.submit()">
+        <p>This is just a dummy page that loads the next page to see if we crash.</p>
+        <form name="myForm" action=""
         <script>
             if (window.testRunner) {
                 testRunner.dumpAsText();
@@ -8,7 +10,5 @@
 
             window.setTimeout("location.href=''", 0);
         </script>
-        <p>This is just a dummy page that loads the next page to see if we crash.</p>
-        <form name="myForm" action=""
     </body>
 <html>
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to