2008-10-12 Pamela Greene <pam@chromium.org>
authorabarth@webkit.org <abarth@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 13 Oct 2008 01:15:32 +0000 (01:15 +0000)
committerabarth@webkit.org <abarth@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 13 Oct 2008 01:15:32 +0000 (01:15 +0000)
        Remove a race condition in a test by promoting layoutTestController
        calls outside document.write().  Add document.close().
        See https://bugs.webkit.org/show_bug.cgi?id=21524

        Reviewed by Darin Adler.
        Landed by Adam Barth.

        * fast/encoding/script-in-head.html:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@37547 268f45cc-cd09-0410-ab3c-d52691b4dbfc

LayoutTests/ChangeLog
LayoutTests/fast/encoding/script-in-head.html

index a861dba..6b7f1d6 100644 (file)
@@ -1,5 +1,16 @@
 2008-10-12  Pamela Greene  <pam@chromium.org>
 
+        Remove a race condition in a test by promoting layoutTestController
+        calls outside document.write().  Add document.close().
+        See https://bugs.webkit.org/show_bug.cgi?id=21524
+
+        Reviewed by Darin Adler.
+        Landed by Adam Barth.
+
+        * fast/encoding/script-in-head.html:
+
+2008-10-12  Pamela Greene  <pam@chromium.org>
+
         Remove data: URLs to avoid cross-domain security restrictions.
         See https://bugs.webkit.org/show_bug.cgi?id=21528
 
index dfb8076..d6dc1ca 100644 (file)
@@ -1,9 +1,9 @@
 <!-- temp --><script language="JavaScript" type="text/JavaScript">
+if (window.layoutTestController) {
+    layoutTestController.dumpAsText();
+    layoutTestController.waitUntilDone();
+}
 document.write ('<SCR' + 'IPT>' +
-    'if (window.layoutTestController) {' +
-        'layoutTestController.dumpAsText();' +
-        'layoutTestController.waitUntilDone();' +
-    '}' +
 
     'setTimeout(function () {' +
         <!-- The letters CCE below are Cyrillic, so we do test that the encoding is correct. -->
@@ -12,7 +12,7 @@ document.write ('<SCR' + 'IPT>' +
         'if (window.layoutTestController)' +
             'layoutTestController.notifyDone();' +
     '}, 0);' +
-'</SCR' + 'IPT>'); </script>
+'</SCR' + 'IPT>');document.close(); </script>
 <!-- /temp --><!--HEADER-->
 
 <!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN"