Bug 453969 - Fix the race in test_bug382113.html so we don't set our …

…child-onload-fired boolean to false _after_ the child onload has already fired. r=froydnj, a=test-only
mozilla · Mar 10, 2015 · 71996a5 · 71996a5
1 parent 1b97832
commit 71996a5
Showing 1 changed file with 13 additions and 16 deletions.
diff --git a/content/base/test/test_bug382113.html b/content/base/test/test_bug382113.html
@@ -7,6 +7,19 @@
   <title>Test for Bug 382113</title>
   <script type="text/javascript" src="/tests/SimpleTest/SimpleTest.js"></script>        
   <link rel="stylesheet" type="text/css" href="/tests/SimpleTest/test.css" />
+  <script class="testbody" type="application/javascript">
+
+    SimpleTest.waitForExplicitFinish();
+    var childGotOnload = false;
+    var objectGotOnload = false;
+
+    /** Test for Bug 100533 **/
+    function checkEvents() {
+      is(childGotOnload, true, "Child got load event");
+      is(objectGotOnload, true, "Object got load event");
+      SimpleTest.finish();
+    }
+  </script>
 </head>
 <body onload="checkEvents()">
 <a target="_blank" href="https://bugzilla.mozilla.org/show_bug.cgi?id=382113">Mozilla Bug 382113</a>
@@ -16,22 +29,6 @@
           onload="objectGotOnload = true;"></object>
 </div>
 <pre id="test">
-
-
-<script class="testbody" type="application/javascript">
-
-SimpleTest.waitForExplicitFinish();
-var childGotOnload = false;
-var objectGotOnload = false;
-
-/** Test for Bug 100533 **/
-function checkEvents() {
-  is(childGotOnload, true, "Child got load event");
-  is(objectGotOnload, true, "Object got load event");
-  SimpleTest.finish();
-}
-</script>
-
 </pre>
 </body>
 </html>