From 2e9f0702f285125a3126930420b697ade34b74dc Mon Sep 17 00:00:00 2001
From: Kevin McNee <mcnee@chromium.org>
Date: Tue, 10 Dec 2024 18:59:56 -0800
Subject: [PATCH] Add WPTs for RelOpenerBcgDependencyHint

Add tests which confirm that rel=opener on anchors and an opener window
feature with window.open() prevent proactive browsing context group
changes. These tests are tentative, since the associated proposal has
not shipped.

These tests are implemented by using a named window lookup after a
session history navigation to verify two pages are in the same
BCG (since the lookup is scoped to BCGs).

Also set RelOpenerBcgDependencyHint's status to experimental. The
feature is needed for these tests to pass.

https://github.com/explainers-by-googlers/future-browsing-context-group-dependency-hint
https://chromestatus.com/feature/5182336170983424

Bug: 333743493
Change-Id: Ibcda5e87fa63b2f9dac55c1382f3e2bc8f2fd711
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6077548
Reviewed-by: Domenic Denicola <domenic@chromium.org>
Commit-Queue: Kevin McNee <mcnee@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1394650}
---
 ...owsing-context-group-change.tentative.html | 92 +++++++++++++++++++
 1 file changed, 92 insertions(+)
 create mode 100644 html/semantics/links/links-created-by-a-and-area-elements/rel-opener-prevents-browsing-context-group-change.tentative.html
diff --git a/html/semantics/links/links-created-by-a-and-area-elements/rel-opener-prevents-browsing-context-group-change.tentative.html b/html/semantics/links/links-created-by-a-and-area-elements/rel-opener-prevents-browsing-context-group-change.tentative.html
new file mode 100644
index 00000000000000..94a9c2d3944c25
--- /dev/null
+++ b/html/semantics/links/links-created-by-a-and-area-elements/rel-opener-prevents-browsing-context-group-change.tentative.html
@@ -0,0 +1,92 @@
+<!DOCTYPE html>
+<title>Prevent browsing context group changes</title>
+<script src="/resources/testharness.js"></script>
+<script src="/resources/testharnessreport.js"></script>
+<script src="/common/utils.js"></script>
+<script src="/common/dispatcher/dispatcher.js"></script>
+<script src="/html/browsers/browsing-the-web/remote-context-helper/resources/remote-context-helper.js"></script>
+
+<body>
+<script>
+
+/*
+ * These tests are tentative. They are based on the following proposal:
+ * https://github.com/explainers-by-googlers/future-browsing-context-group-dependency-hint
+ */
+
+function navigateByRelOpenerAnchorToNew(remote) {
+  function navigateByRelOpenerAnchorExecutorCreator(remote) {
+    return (url) => {
+      return remote.navigate((url) => {
+        let anchor = document.createElement('a');
+        anchor.href = url;
+        anchor.rel = 'opener';
+        anchor.text = 'Link';
+        document.body.appendChild(anchor);
+        anchor.click();
+      }, [url]);
+    };
+  }
+
+  return remote.helper.createContext({
+        executorCreator: navigateByRelOpenerAnchorExecutorCreator(remote),
+      });
+}
+
+function navigateBySelfOpenToNew(remote) {
+  function navigateBySelfOpenExecutorCreator(remote) {
+    return (url) => {
+      return remote.navigate((url) => {
+        window.open(url, '_self', 'opener');
+      }, [url]);
+    };
+  }
+
+  return remote.helper.createContext({
+        executorCreator: navigateBySelfOpenExecutorCreator(remote),
+      });
+}
+
+async function runNoGroupChangeTest(performNavigation) {
+  const rcHelper = new RemoteContextHelper();
+  // This test harness cannot be in the same browsing context group as the
+  // navigating window, otherwise that would interfere with the test.
+  const rcWindow = await rcHelper.addWindow(undefined, { features: 'noopener' });
+
+  const rcWindow2 = await performNavigation(rcWindow);
+
+  const rcPopup = await rcWindow2.addWindow(undefined, { target: 'my_popup' });
+
+  assert_equals(await rcPopup.executeScript(() => {
+    window.previouslyOpened = true;
+    return window.name;
+  }), 'my_popup', 'popup created');
+
+  rcWindow2.historyBack();
+
+  // In order for the original page to find the popup by name, they must be in
+  // the same browsing context group.
+  assert_true(await rcWindow.executeScript(() => {
+    const popup = window.open('', 'my_popup');
+    return popup.previouslyOpened;
+  }), 'first page can access original popup');
+}
+
+promise_test(async t => {
+  await runNoGroupChangeTest((remote) => {
+    // Navigate away using rel=opener. The next page should remain in the same
+    // browsing context group.
+    return navigateByRelOpenerAnchorToNew(remote);
+  });
+}, 'rel=opener prevents browsing context group change');
+
+promise_test(async t => {
+  await runNoGroupChangeTest((remote) => {
+    // Navigate away using window.open with the opener window feature. The next
+    // page should remain in the same browsing context group.
+    return navigateBySelfOpenToNew(remote);
+  });
+}, 'opener window feature prevents browsing context group change');
+
+</script>
+</body>
