Closed Bug 1867900 Opened 2 years ago Closed 2 years ago

[wpt-sync] Sync PR 43487 - [navigation API] Fix flaky scroll restoration test

Categories

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

task

Tracking

(firefox122 fixed)

RESOLVED FIXED
122 Branch
Tracking Status
firefox122 --- fixed

People

(Reporter: wpt-sync, Unassigned)

References

()

Details

(Whiteboard: [wptsync downstream])

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

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

Scott Haseley <shaseley@chromium.org> wrote:

[navigation API] Fix flaky scroll restoration test

after-transition-intercept-handler-modifies.html sometimes fails
because the scroll restoration is being overridden if the layout
changes from the previous same document navigation have not yet been
applied. This becomes more consistent when same document navigation IPCs
are prioritized (see crrev.com/c/4981177).

This CL changes the test to ensure rendering happens between the forward
and back navigations, which in turn ensures the new layout and scroll
position were updated before navigating back.

Bug: 1042118
Change-Id: Ia6a3768f8780f9ddaff80f7fd877ddca5c6184b7

Reviewed-on: https://chromium-review.googlesource.com/5080962
WPT-Export-Revision: 08412e456f483b958e63476c7c162c46b1b11cda

CI Results

Ran 0 Firefox configurations based on mozilla-central, and Firefox, Chrome, and Safari on GitHub CI

Total 1 tests and 1 subtests

Status Summary

Firefox

OK : 1
FAIL: 1

Chrome

OK : 1
FAIL: 1

Safari

OK : 1
FAIL: 1

Links

GitHub PR Head
GitHub PR Base

Details

New Tests That Don't Pass

Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/037026d87224 [wpt PR 43487] - [navigation API] Fix flaky scroll restoration test, a=testonly
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 122 Branch
You need to log in before you can comment on or make changes to this bug.