Closed Bug 1849637 Opened 2 years ago Closed 2 years ago

[wpt-sync] Sync PR 41569 - Remove scrollend timeouts on user scroll tests

Categories

(Core :: DOM: Core & HTML, task, P4)

task

Tracking

()

RESOLVED FIXED
119 Branch
Tracking Status
firefox119 --- fixed

People

(Reporter: wpt-sync, Unassigned)

References

()

Details

(Whiteboard: [wptsync downstream])

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

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

David Awogbemila <awogbemila@chromium.org> wrote:

Remove scrollend timeouts on user scroll tests

Enforcing timeouts while waiting for scrollend can cause flakiness
in tests, especially for tests that perform user scrolls.

This patch introduces a waitForScrollendNoTimeout function which tests
can use to wait for scrollend events without enforcing arbitrary
timeouts. The plan is to replace the timeout-enforcing variant in all
the WPTs. This patch starts with user scroll tests which are
particularly sensitive to the timeouts.

Bug: 1469445
Change-Id: I80c82017a308f779f2b3388039e84cfc452a3503
Reviewed-on: https://chromium-review.googlesource.com/4799689
WPT-Export-Revision: 23b8d5c51fcb563521d391c2c95c930a2e9f5c7b

Component: web-platform-tests → DOM: Core & HTML
Product: Testing → Core
Test result changes from PR not available.
Test result changes from PR not available.
Test result changes from PR not available.
Test result changes from PR not available.
Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/2b8eb2160581 [wpt PR 41569] - Remove scrollend timeouts on user scroll tests, a=testonly https://hg.mozilla.org/integration/autoland/rev/a96e62d2e53e [wpt PR 41569] - Update wpt metadata, a=testonly
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 119 Branch
You need to log in before you can comment on or make changes to this bug.