Skip to content

Reland "Prerender: Add WPTs for prerendering for URLs with fragments" #41174

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

chromium-wpt-export-bot
Copy link
Collaborator

@chromium-wpt-export-bot chromium-wpt-export-bot commented Jul 25, 2023

This is a reland of commit e4a8214765c98b0791f183905dd7eeb63be499a1

Fix the main cause of flakiness (The arrival order of postMessage is
non deterministic).
Please see the diff of ps2 and ps3.

Original change's description:

Prerender: Add WPTs for prerendering for URLs with fragments

This CL adds WPTs to test and clarify current behavior of speculation-
rules triggered prerendering for URLs with fragments:

  • If each URL specified for prerender are not completely identical, it
    will be prerendered individually. Therefore, even if some URLs differ
    only in fragments, they will be prerendered separately.
  • The URL of the activated page should match the prerendering URL so
    that they should contain the same fragments.

Bug: 1417936
Change-Id: I896709b25b26562919785b784a3d8b7f2bb384d8
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4478530
Commit-Queue: Taiyo Mizuhashi <taiyo@chromium.org>
Reviewed-by: Hiroki Nakagawa <nhiroki@chromium.org>
Reviewed-by: Huanpo Lin <robertlin@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1141291}

Bug: 1417936
Change-Id: I312ab1c29ccf67af50c2ce811371ae89c74ec07c

Reviewed-on: https://chromium-review.googlesource.com/4716019
WPT-Export-Revision: d9bc1e571e02f487b65c540a78e5d074e794bce5

Copy link
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.

@chromium-wpt-export-bot chromium-wpt-export-bot force-pushed the chromium-export-cl-4716019 branch 2 times, most recently from ee988f9 to 27c3093 Compare August 2, 2023 10:12
@chromium-wpt-export-bot chromium-wpt-export-bot force-pushed the chromium-export-cl-4716019 branch 3 times, most recently from 3abbd33 to aaa6934 Compare January 15, 2024 14:08
This is a reland of commit e4a8214765c98b0791f183905dd7eeb63be499a1

Fix the main cause of flakiness (The arrival order of postMessage is
non deterministic).
Please see the diff of ps2 and ps3.

Original change's description:
> Prerender: Add WPTs for prerendering for URLs with fragments
>
> This CL adds WPTs to test and clarify current behavior of speculation-
> rules triggered prerendering for URLs with fragments:
> - If each URL specified for prerender are not completely identical, it
>   will be prerendered individually. Therefore, even if some URLs differ
>   only in fragments, they will be prerendered separately.
> - The URL of the activated page should match the prerendering URL so
>   that they should contain the same fragments.
>
> Bug: 1417936
> Change-Id: I896709b25b26562919785b784a3d8b7f2bb384d8
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4478530
> Commit-Queue: Taiyo Mizuhashi <taiyo@chromium.org>
> Reviewed-by: Hiroki Nakagawa <nhiroki@chromium.org>
> Reviewed-by: Huanpo Lin <robertlin@chromium.org>
> Cr-Commit-Position: refs/heads/main@{#1141291}

Bug: 1417936
Change-Id: I312ab1c29ccf67af50c2ce811371ae89c74ec07c
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.

2 participants