Closed Bug 1909469 Opened 3 months ago Closed 3 months ago

[wpt-sync] Sync PR 47260 - Do not fire popstate during No-Vary-Search prerender activation

Categories

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

Product:
Testing
Component:
web-platform-tests
Type:
task

Tracking

(firefox130 fixed)

Status:
RESOLVED FIXED
Milestone:
130 Branch
Tracking Flags:
Tracking Status
firefox130 --- fixed

People

(Reporter: wpt-sync, Unassigned)

References

(
URL
)

Details

(Whiteboard: [wptsync downstream])

Attachments

(5 files)

Notable wpt changes
64.82 KB, text/plain
Details
Notable wpt changes
64.82 KB, text/plain
Details
Notable wpt changes
64.82 KB, text/plain
Details
Notable wpt changes
64.82 KB, text/plain
Details
Notable wpt changes
64.82 KB, text/plain
Details

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

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

Domenic Denicola <domenic@chromium.org> wrote:

Do not fire popstate during No-Vary-Search prerender activation

Doing so is against the spec, and only happened accidentally due to a quirk in how Chromium implements the spec's "URL and history update steps".

This also adds a bit of extra test coverage that the URLs (as exposed by location.href) actually change.

Fixed: 354748055
Change-Id: Ic528cb383574809a8e909d141209f8ae6c761323
Reviewed-on: https://chromium-review.googlesource.com/5733313
WPT-Export-Revision: 680ddcae206d72bf5fcaf52a4a53551e78bc7860

Attached file Notable wpt changes —
# CI Results Ran 9 Firefox configurations based on mozilla-central, and Firefox, Chrome, and Safari on GitHub CI Total 207 tests ## Status Summary ### Firefox `FAIL` : 2 `TIMEOUT`: 2 `ERROR` : 204 `NOTRUN` : 3 ### Chrome `OK` : 169 `PASS` : 160 `FAIL` : 35 `TIMEOUT`: 74 `NOTRUN` : 5 ### Safari `FAIL` : 2 `TIMEOUT`: 2 `ERROR` : 204 `NOTRUN` : 3 ## Links [Gecko CI (Treeherder)](https://treeherder.mozilla.org/#/jobs?repo=try&revision=42459b38a758fb5e9f9926668b01f339df6689c4) [GitHub PR Head](https://wpt.fyi/results/?sha=8ffb27caa96c71c2ab0c33b0e072501dc5a1a0a0&label=pr_head) [GitHub PR Base](https://wpt.fyi/results/?sha=8ffb27caa96c71c2ab0c33b0e072501dc5a1a0a0&label=pr_base) ## Details (See attachment for full changes)
Attached file Notable wpt changes —
# CI Results Ran 9 Firefox configurations based on mozilla-central, and Firefox, Chrome, and Safari on GitHub CI Total 207 tests ## Status Summary ### Firefox `FAIL` : 2 `TIMEOUT`: 2 `ERROR` : 204 `NOTRUN` : 3 ### Chrome `OK` : 169 `PASS` : 160 `FAIL` : 35 `TIMEOUT`: 74 `NOTRUN` : 5 ### Safari `FAIL` : 2 `TIMEOUT`: 2 `ERROR` : 204 `NOTRUN` : 3 ## Links [Gecko CI (Treeherder)](https://treeherder.mozilla.org/#/jobs?repo=try&revision=42459b38a758fb5e9f9926668b01f339df6689c4) [GitHub PR Head](https://wpt.fyi/results/?sha=8ffb27caa96c71c2ab0c33b0e072501dc5a1a0a0&label=pr_head) [GitHub PR Base](https://wpt.fyi/results/?sha=8ffb27caa96c71c2ab0c33b0e072501dc5a1a0a0&label=pr_base) ## Details (See attachment for full changes)
Attached file Notable wpt changes —
# CI Results Ran 9 Firefox configurations based on mozilla-central, and Firefox, Chrome, and Safari on GitHub CI Total 207 tests ## Status Summary ### Firefox `FAIL` : 2 `TIMEOUT`: 2 `ERROR` : 204 `NOTRUN` : 3 ### Chrome `OK` : 169 `PASS` : 160 `FAIL` : 35 `TIMEOUT`: 74 `NOTRUN` : 5 ### Safari `FAIL` : 2 `TIMEOUT`: 2 `ERROR` : 204 `NOTRUN` : 3 ## Links [Gecko CI (Treeherder)](https://treeherder.mozilla.org/#/jobs?repo=try&revision=42459b38a758fb5e9f9926668b01f339df6689c4) [GitHub PR Head](https://wpt.fyi/results/?sha=8ffb27caa96c71c2ab0c33b0e072501dc5a1a0a0&label=pr_head) [GitHub PR Base](https://wpt.fyi/results/?sha=8ffb27caa96c71c2ab0c33b0e072501dc5a1a0a0&label=pr_base) ## Details (See attachment for full changes)
Attached file Notable wpt changes —
# CI Results Ran 9 Firefox configurations based on mozilla-central, and Firefox, Chrome, and Safari on GitHub CI Total 207 tests ## Status Summary ### Firefox `FAIL` : 2 `TIMEOUT`: 2 `ERROR` : 204 `NOTRUN` : 3 ### Chrome `OK` : 169 `PASS` : 160 `FAIL` : 35 `TIMEOUT`: 74 `NOTRUN` : 5 ### Safari `FAIL` : 2 `TIMEOUT`: 2 `ERROR` : 204 `NOTRUN` : 3 ## Links [Gecko CI (Treeherder)](https://treeherder.mozilla.org/#/jobs?repo=try&revision=42459b38a758fb5e9f9926668b01f339df6689c4) [GitHub PR Head](https://wpt.fyi/results/?sha=8ffb27caa96c71c2ab0c33b0e072501dc5a1a0a0&label=pr_head) [GitHub PR Base](https://wpt.fyi/results/?sha=8ffb27caa96c71c2ab0c33b0e072501dc5a1a0a0&label=pr_base) ## Details (See attachment for full changes)
Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/4aa0c1ae5100 [wpt PR 47260] - Do not fire popstate during No-Vary-Search prerender activation, a=testonly
Attached file Notable wpt changes —
# CI Results Ran 9 Firefox configurations based on mozilla-central, and Firefox, Chrome, and Safari on GitHub CI Total 207 tests ## Status Summary ### Firefox `FAIL` : 2 `TIMEOUT`: 2 `ERROR` : 204 `NOTRUN` : 3 ### Chrome `OK` : 169 `PASS` : 160 `FAIL` : 35 `TIMEOUT`: 74 `NOTRUN` : 5 ### Safari `FAIL` : 2 `TIMEOUT`: 2 `ERROR` : 204 `NOTRUN` : 3 ## Links [Gecko CI (Treeherder)](https://treeherder.mozilla.org/#/jobs?repo=try&revision=42459b38a758fb5e9f9926668b01f339df6689c4) [GitHub PR Head](https://wpt.fyi/results/?sha=8ffb27caa96c71c2ab0c33b0e072501dc5a1a0a0&label=pr_head) [GitHub PR Base](https://wpt.fyi/results/?sha=8ffb27caa96c71c2ab0c33b0e072501dc5a1a0a0&label=pr_base) ## Details (See attachment for full changes)
Test result changes from PR not available.

https://hg.mozilla.org/mozilla-central/rev/4aa0c1ae5100

Status: NEW → RESOLVED
Closed: 3 months ago
status-firefox130: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 130 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: