Avoid timeout in iframe loading=lazy navigation.reload test

zcorpan · web-flow · commit 99e8d52f36fd · 2025-05-14T17:06:21.000+02:00
If `navigation.reload` is not supported, the `iframe.hidden = false;` line was not reached, so `iframeLoaded` would never resolve.
diff --git a/html/semantics/embedded-content/the-iframe-element/iframe-loading-lazy-nav-navigation-navigate.html b/html/semantics/embedded-content/the-iframe-element/iframe-loading-lazy-nav-navigation-navigate.html
@@ -14,8 +14,8 @@
 <script src="/resources/testharnessreport.js"></script>
 <script>
 setup({single_test: true});
+assert_true("navigation" in window, "Navigation API is supported");
 iframeLoaded.then(() => {
-  assert_true("navigation" in window, "Navigation API is supported");
   // Need a timeout to detect failure when there are two navigations.
   step_timeout(() => {
     assert_equals(iframe.contentWindow.location.href, new URL("support/blank.htm?nav", location.href).href);
diff --git a/html/semantics/embedded-content/the-iframe-element/iframe-loading-lazy-reload-navigation-reload.html b/html/semantics/embedded-content/the-iframe-element/iframe-loading-lazy-reload-navigation-reload.html
@@ -1,5 +1,5 @@
 <!DOCTYPE html>
-<title>Reloading iframe loading='lazy' before it is loaded: location.reload</title>
+<title>Reloading iframe loading='lazy' before it is loaded: navigation.reload</title>
 <iframe src="support/blank.htm?src" loading="lazy" hidden></iframe>
 <script>
 const iframe = document.querySelector('iframe');
@@ -15,10 +15,10 @@
 <script src="/resources/testharnessreport.js"></script>
 <script>
 setup({single_test: true});
+assert_true("navigation" in window, "Navigation API is supported");
 iframeLoaded.then(() => {
   // Need a timeout to detect failure when there are two navigations.
   step_timeout(() => {
-    assert_true("navigation" in window, "Navigation API is supported");
     assert_equals(iframe.contentWindow.location.href, new URL("support/blank.htm?src", location.href).href);
     done();
   }, 1000);
