From 1d11535c45643819bfccf41c621eea2974df3171 Mon Sep 17 00:00:00 2001
From: Dominic Farolino <dom@chromium.org>
Date: Tue, 9 Apr 2024 07:08:20 -0700
Subject: [PATCH] Fix `<meta>` referrer post-insertion steps expectations

See https://github.com/whatwg/html/issues/10241 for the recommendation
to change these test expectations to align with Chromium and WebKit's
behavior.

R=nrosenthal@chromium.org

Bug: 40150299
Change-Id: I3cabc92c16a4cfadc6675bd156aca04c2ade64aa
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5420410
Commit-Queue: Dominic Farolino <dom@chromium.org>
Reviewed-by: Noam Rosenthal <nrosenthal@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1284444}
---
 ...er-and-script-from-fragment.tentative.html | 54 +++++++++++--------
 1 file changed, 33 insertions(+), 21 deletions(-)

diff --git a/dom/nodes/insertion-removing-steps/Node-append-meta-referrer-and-script-from-fragment.tentative.html b/dom/nodes/insertion-removing-steps/Node-append-meta-referrer-and-script-from-fragment.tentative.html
index d2477976033762..e80e3b45b61d99 100644
--- a/dom/nodes/insertion-removing-steps/Node-append-meta-referrer-and-script-from-fragment.tentative.html
+++ b/dom/nodes/insertion-removing-steps/Node-append-meta-referrer-and-script-from-fragment.tentative.html
@@ -5,25 +5,37 @@
 <script src=/resources/testharnessreport.js></script>
 <script>
 promise_test(async t => {
-    const script = document.createElement("script");
-    const meta = document.createElement("meta");
-    meta.name = "referrer";
-    meta.content = "no-referrer";
-    const fragment = new DocumentFragment();
-    const done = new Promise(resolve => {
-        window.didFetch = resolve;
-    });
-    script.textContent = `
-        (async function() {
-            const response = await fetch("/html/infrastructure/urls/terminology-0/resources/echo-referrer-text.py")
-            const text = await response.text();
-            window.didFetch(text);
-        })();
-    `;
-    fragment.append(script, meta);
-    document.head.append(fragment);
-    const result = await done;
-    assert_equals(result, "");
-}, "<meta name=referrer> should apply before script, as it is an insertion step " +
-    "and not a post-insertion step");
+  const preMetaScript = document.createElement("script");
+  preMetaScript.textContent = `
+    window.preMetaScriptPromise = fetch('/html/infrastructure/urls/terminology-0/resources/echo-referrer-text.py')
+      .then(response => response.text());
+  `;
+
+  const meta = document.createElement("meta");
+  meta.name = "referrer";
+  meta.content = "no-referrer";
+
+  const postMetaScript = document.createElement("script");
+  postMetaScript.textContent = `
+    window.postMetaScriptPromise = fetch('/html/infrastructure/urls/terminology-0/resources/echo-referrer-text.py')
+      .then(response => response.text());
+  `;
+
+  const fragment = new DocumentFragment();
+  fragment.append(preMetaScript, meta, postMetaScript);
+  document.head.append(fragment);
+
+  const preMetaReferrer = await window.preMetaScriptPromise;
+  assert_equals(preMetaReferrer, location.href,
+      "preMetaReferrer is the full URL; by the time the first script runs in " +
+      "its post-insertion steps, the later-inserted meta tag has not run its " +
+      "post-insertion steps, which is where meta tags are processed");
+
+  const postMetaReferrer = await window.postMetaScriptPromise;
+  assert_equals(postMetaReferrer, "",
+      "postMetaReferrer is empty; by the time the second script runs in " +
+      "its post-insertion steps, the later-inserted meta tag has run its " +
+      "post-insertion steps, and observes the meta tag's effect");
+}, "<meta name=referrer> gets processed and applied in the post-insertion " +
+   "steps");
 </script>