Test user clicks on <a>s

web-platform-tests · May 21, 2021 · 31dccb1 · 31dccb1
1 parent 97fc6b0
commit 31dccb1
Show file tree

Hide file tree

Showing 5 changed files with 87 additions and 1 deletion.
diff --git a/...ing-the-web/navigating-across-documents/replace-before-load/a-user-click-during-load.html b/...ing-the-web/navigating-across-documents/replace-before-load/a-user-click-during-load.html
@@ -0,0 +1,36 @@
+<!DOCTYPE html>
+<meta charset="utf-8">
+<script src="/resources/testharness.js"></script>
+<script src="/resources/testharnessreport.js"></script>
+<script src="resources/helpers.js"></script>
+<script src="/resources/testdriver.js"></script>
+<script src="/resources/testdriver-vendor.js"></script>
+
+<body>
+<script>
+"use strict";
+promise_test(async t => {
+  const sentinelIframe = await setupSentinelIframe(t);
+  const startingHistoryLength = history.length;
+
+  const code = `
+    window.onload = () => {
+      const a = document.createElement("a");
+      a.href = "/common/blank.html?thereplacement";
+      a.id = "the-anchor";
+      a.textContent = "needs to have content to be clickable";
+      document.body.append(a);
+      parent.test_driver.click(a);
+    };
+  `;
+
+  const startURL = "resources/slow-code-injector.html?pipe=sub(none)&code=" + encodeURIComponent(code);
+  const afterReplacementURL = "/common/blank.html?thereplacement";
+  const iframe = insertIframe(t, startURL);
+
+  assert_equals(history.length, startingHistoryLength, "Inserting the under-test iframe must not change history.length");
+
+  await waitForLoadAllowingIntermediateLoads(t, iframe, afterReplacementURL);
+  assert_equals(history.length, startingHistoryLength + 1, "history.length must change after waiting for the load");
+}, "User click on <a> during the load event must NOT replace");
+</script>
diff --git a/...owsers/browsing-the-web/navigating-across-documents/replace-before-load/a-user-click.html b/...owsers/browsing-the-web/navigating-across-documents/replace-before-load/a-user-click.html
@@ -0,0 +1,34 @@
+<!DOCTYPE html>
+<meta charset="utf-8">
+<script src="/resources/testharness.js"></script>
+<script src="/resources/testharnessreport.js"></script>
+<script src="resources/helpers.js"></script>
+<script src="/resources/testdriver.js"></script>
+<script src="/resources/testdriver-vendor.js"></script>
+
+<body>
+<script>
+"use strict";
+promise_test(async t => {
+  const sentinelIframe = await setupSentinelIframe(t);
+  const startingHistoryLength = history.length;
+
+  const code = `
+    const a = document.createElement("a");
+    a.href = "/common/blank.html?thereplacement";
+    a.id = "the-anchor";
+    a.textContent = "needs to have content to be clickable";
+    document.currentScript.before(a);
+    parent.test_driver.click(a);
+  `;
+
+  const startURL = "resources/slow-code-injector.html?pipe=sub(none)&code=" + encodeURIComponent(code);
+  const afterReplacementURL = "/common/blank.html?thereplacement";
+  const iframe = insertIframe(t, startURL);
+
+  assert_equals(history.length, startingHistoryLength, "Inserting the under-test iframe must not change history.length");
+
+  await waitForLoad(t, iframe, afterReplacementURL);
+  assert_equals(history.length, startingHistoryLength + 1, "history.length must change after waiting for the load");
+}, "User click on <a> before the load event must NOT replace");
+</script>
diff --git a/...sing-the-web/navigating-across-documents/replace-before-load/resources/code-injector.html b/...sing-the-web/navigating-across-documents/replace-before-load/resources/code-injector.html
@@ -2,6 +2,7 @@
 <meta charset="utf-8">
 <title>Subframe</title>
 
+<body>
 <script>
 "use strict";
 {{GET[code]}}

diff --git a/...ers/browsing-the-web/navigating-across-documents/replace-before-load/resources/helpers.js b/...ers/browsing-the-web/navigating-across-documents/replace-before-load/resources/helpers.js
@@ -50,6 +50,9 @@ window.insertIframe = (t, url) => {
   const iframe = document.createElement("iframe");
   iframe.src = url;
   document.body.append(iframe);
-  t.add_cleanup(() => iframe.remove());
+
+  // Intentionally not including the following:
+  //  t.add_cleanup(() => iframe.remove());
+  // Doing so breaks some of the testdriver.js tests with "cannot find window" errors.
   return iframe;
 };
diff --git a/...the-web/navigating-across-documents/replace-before-load/resources/slow-code-injector.html b/...the-web/navigating-across-documents/replace-before-load/resources/slow-code-injector.html
@@ -0,0 +1,12 @@
+<!DOCTYPE html>
+<meta charset="utf-8">
+<title>Subframe</title>
+
+<body>
+<script>
+"use strict";
+{{GET[code]}}
+</script>
+
+<!-- This is necessary in cases involving user interaction because those happen async through the webdriver infrastructure -->
+<img src="/common/slow.py">