Closed Bug 1832207 Opened 2 years ago Closed 2 years ago

[wpt-sync] Sync PR 39918 - Revert "Prerender: Add WPTs for prerendering for URLs with fragments"

Categories

(Testing :: web-platform-tests, task, P4)

task

Tracking

(firefox115 fixed)

RESOLVED FIXED
115 Branch
Tracking Status
firefox115 --- fixed

People

(Reporter: wpt-sync, Unassigned)

References

()

Details

(Whiteboard: [wptsync downstream])

Sync web-platform-tests PR 39918 into mozilla-central (this bug is closed when the sync is complete).

PR: https://github.com/web-platform-tests/wpt/pull/39918
Details from upstream follow.

Nidhi Jaju <nidhijaju@chromium.org> wrote:

Revert "Prerender: Add WPTs for prerendering for URLs with fragments"

This reverts commit e4a8214765c98b0791f183905dd7eeb63be499a1.

Reason for revert: This is causing failures on https://ci.chromium.org/p/chromium/builders/ci/mac12-arm64-rel-tests

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: I47acf6b8c596e1ba210887e4d68f7b56dd8ff588
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4520356
Reviewed-by: Nidhi Jaju \<nidhijaju@chromium.org>
Commit-Queue: Nidhi Jaju \<nidhijaju@chromium.org>
Commit-Queue: Rubber Stamper \<rubber-stamper@appspot.gserviceaccount.com>
Bot-Commit: Rubber Stamper \<rubber-stamper@appspot.gserviceaccount.com>
Owners-Override: Nidhi Jaju \<nidhijaju@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1141798}

PR 39918 applied with additional changes from upstream: 1d502833eb0ee61372d0d7322ebe4fc77e2ecd1f
Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/764584a3d2df [wpt PR 39918] - Revert "Prerender: Add WPTs for prerendering for URLs with fragments", a=testonly
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 115 Branch
You need to log in before you can comment on or make changes to this bug.