Bug 1386391 - Poll to check if a document has loaded rather than a definite timeout. r=smaug

MozReview-Commit-ID: B293QZe3LIz --HG-- extra : rebase_source : ae336c328b48799a3a952f8d14cb8532fc179a58 extra : source : ee4a89243a26456b495737d889402619d3e186cc
2017-08-14 10:34:04 +08:00 · 2017-08-14 10:34:04 +08:00 · 9e499caed6
--- a/docshell/test/navigation/file_fragment_handling_during_load.html
+++ b/docshell/test/navigation/file_fragment_handling_during_load.html
@ -1,22 +1,24 @@
 <html>
  <head>
    <script>
-      var timerID = 0;
-      function testDone() {
-        clearTimeout(timerID);
-        var l = document.body.firstChild.contentWindow.location.href;
-        opener.ok(l.endsWith("file_fragment_handling_during_load_frame2.html"),
-                 "Should have loaded a new document");
+      function checkHaveLoadedNewDoc() {
+        let l = document.body.firstChild.contentWindow.location.href;
+        if (!l.endsWith("file_fragment_handling_during_load_frame2.html")) {
+          opener.ok(true, "Fine. We will check later.");
+          setTimeout(checkHaveLoadedNewDoc, 500);
+          return;
+        }
+        opener.ok(true, "Have loaded a new document.");
        opener.nextTest();
        window.close();
      }
      function test() {
        var ifr = document.getElementsByTagName("iframe")[0];
-        ifr.onload = testDone;
+        ifr.onload = checkHaveLoadedNewDoc;
        ifr.contentWindow.location.hash = "b";
        ifr.contentWindow.location.href = "file_fragment_handling_during_load_frame2.html";
        history.back();
-        timerID = setTimeout(testDone, 2000);
+        checkHaveLoadedNewDoc();
      }
    </script>
  </head>