Skip to content

Activation across navigation: add a WPT for cross-origin redirection#59293

Merged
DanielRyanSmith merged 1 commit intomasterfrom
chromium-export-cl-7769229
Apr 29, 2026
Merged

Activation across navigation: add a WPT for cross-origin redirection#59293
DanielRyanSmith merged 1 commit intomasterfrom
chromium-export-cl-7769229

Conversation

@chromium-wpt-export-bot
Copy link
Copy Markdown
Collaborator

@chromium-wpt-export-bot chromium-wpt-export-bot commented Apr 16, 2026

This CL adds a new WPT for user activation states after a cross-origin
redirection, using RemoteContext API in dispatcher.js. This behavior
is not fully defined in the spec [1], and the test was written based on
the behavior expected in a PR discussion [2].

This CL also simplifies the existing same-origin test using the same
API, and fixes the visibility of 3 private methods in utils.js.

[1]
https://html.spec.whatwg.org/multipage/browsing-the-web.html#navigation-and-session-history:~:text=Welcome%20to%20the%20dragon%27s%20maw.
[2] whatwg/html#11454

Bug: 480993679
Change-Id: I9c7e5ad836404d7b1bf8e484a89fc2824b7c0364
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7769229
Commit-Queue: Mustaq Ahmed <mustaq@chromium.org>
Reviewed-by: Vladimir Levin <vmpstr@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1616504}

This CL adds a new WPT for user activation states after a cross-origin
redirection, using `RemoteContext` API in dispatcher.js. This behavior
is not fully defined in the spec [1], and the test was written based on
the behavior expected in a PR discussion [2].

This CL also simplifies the existing same-origin test using the same
API, and fixes the visibility of 3 private methods in utils.js.

[1]
https://html.spec.whatwg.org/multipage/browsing-the-web.html#navigation-and-session-history:~:text=Welcome%20to%20the%20dragon%27s%20maw.
[2] whatwg/html#11454

Bug: 480993679
Change-Id: I9c7e5ad836404d7b1bf8e484a89fc2824b7c0364
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7769229
Commit-Queue: Mustaq Ahmed <mustaq@chromium.org>
Reviewed-by: Vladimir Levin <vmpstr@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1616504}
Copy link
Copy Markdown
Collaborator

@wpt-pr-bot wpt-pr-bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The review process for this patch is being conducted in the Chromium project.

@mustaqahmed
Copy link
Copy Markdown
Member

FYI @annevk and @foolip, this PR should unblock whatwg/html#11454.

@jonathan-j-lee
Copy link
Copy Markdown
Contributor

Looks like touching /common/utils.js causes CI to run too many tests:

116:21.37 TEST_START: /speculation-rules/prerender/referrer-policy-mismatch.https.html

[taskcluster:error] Task timeout after 7200 seconds. Force killing container.
[taskcluster 2026-04-18 02:16:22.197Z] === Task Finished ===
[taskcluster 2026-04-18 02:16:22.197Z] Unsuccessful task run with exit code: -1 completed in 7201.756 seconds

@DanielRyanSmith Can you force merge this please? Thanks.

@DanielRyanSmith DanielRyanSmith merged commit f58c291 into master Apr 29, 2026
28 of 34 checks passed
@DanielRyanSmith DanielRyanSmith deleted the chromium-export-cl-7769229 branch April 29, 2026 20:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants