Closed Bug 1980427 Opened 1 year ago Closed 1 year ago

[wpt-sync] Sync PR 54076 - Refactor wpt snap-at-user-scroll-end test to prevent flakiness

Categories

(Core :: Layout: Scrolling and Overflow, task, P4)

task

Tracking

()

RESOLVED FIXED
143 Branch
Tracking Status
firefox143 --- fixed

People

(Reporter: wpt-sync, Unassigned)

References

()

Details

(Whiteboard: [wptsync downstream])

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

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

John An <johna@microsoft.com> wrote:

Refactor wpt snap-at-user-scroll-end test to prevent flakiness

  • Remove manual steps and UI (html) since test is automated
  • Encapsulate test in promise_test to allow await for async operations.
    As opposed to sequencing events using chained .then(). This improves
    readability.
  • Wait for wheel events before evaluating scroll completion to reduce
    flakiness. Currently test waits for a scroll position to stop
    changing rather than directly testing for scroll end. So the test
    should ensure a wheel event was received before examining the scroll
    position
  • Wait for compositor resources to load before querying for them in
    the javascript.

Bug: 40250245
Change-Id: I97dc80b174d976807c26a1bd058e5f845b6937d2
Reviewed-on: https://chromium-review.googlesource.com/6799109
WPT-Export-Revision: 6abf6a47ae5e67fb1591833bad63bf0d16bb5db0

Component: web-platform-tests → Layout: Scrolling and Overflow
Product: Testing → Core

CI Results

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

Total 186 tests

Status Summary

Firefox

OK : 1[GitHub] 152[Gecko-android-em-7.0-x86_64-lite-qr-opt-geckoview, Gecko-android-em-7.0-x86_64-qr-debug-geckoview, Gecko-android-em-7.0-x86_64-qr-opt-geckoview] 153[Gecko-linux2404-64-opt] 154[Gecko-linux2404-64-debug, Gecko-windows11-32-24h2-debug, Gecko-windows11-32-24h2-opt, Gecko-windows11-64-24h2-debug, Gecko-windows11-64-24h2-opt]
PASS : 2[Gecko-linux2204-64-wayland-debug, Gecko-linux2204-64-wayland-opt] 682[Gecko-windows11-32-24h2-opt] 683[Gecko-windows11-32-24h2-debug] 699[Gecko-linux2404-64-opt] 701[Gecko-linux2404-64-debug, Gecko-windows11-64-24h2-opt] 702[Gecko-windows11-64-24h2-debug] 703[Gecko-android-em-7.0-x86_64-lite-qr-opt-geckoview, Gecko-android-em-7.0-x86_64-qr-debug-geckoview, Gecko-android-em-7.0-x86_64-qr-opt-geckoview]
FAIL : 1[GitHub] 282[Gecko-android-em-7.0-x86_64-lite-qr-opt-geckoview, Gecko-android-em-7.0-x86_64-qr-debug-geckoview, Gecko-android-em-7.0-x86_64-qr-opt-geckoview] 283[Gecko-windows11-32-24h2-debug] 284[Gecko-windows11-32-24h2-opt] 286[Gecko-windows11-64-24h2-debug] 287[Gecko-linux2404-64-debug, Gecko-windows11-64-24h2-opt] 288[Gecko-linux2404-64-opt]
TIMEOUT: 2

Chrome

OK : 1
PASS : 1

Safari

TIMEOUT: 2

Links

Gecko CI (Treeherder)
GitHub PR Head
GitHub PR Base

Details

New Tests That Don't Pass

Tests Disabled in Gecko Infrastructure

Status: NEW → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 143 Branch
You need to log in before you can comment on or make changes to this bug.