1832482 - [wpt-sync] Sync PR 39954 - Remove step_timeout workaround when waiting for scroll coordinates to update

Status: RESOLVED FIXED

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

Description

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

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

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

Tim Nguyen <[e-mail]> wrote:

Remove step_timeout workaround when waiting for scroll coordinates to update

The pattern addEventListener("scroll", () => step_timeout(resolve, 0)) was used for waiting for scroll coordinates to update after a scroll was performed as the result of a focus() call. step_timeout(resolve, 0) was notably needed because WebKit had a bug where the scroll coordinates (e.g. scrollTop) wouldn't update at the time the scroll event was emitted.

However that bug looks fixed in recent WebKit builds, so this workaround can be removed, and we can use the following instead:

addEventListener("scroll", resolve)

Comment 1

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=0cd8624ef9767edd3edf8f16ee88e34cb27d4d2d

Comment 2

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=17c646e24e9a8eb80aa3cc1509ab4ed07c3a531b

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/e184c11621a6
[wpt PR 39954] - Remove step_timeout workaround when waiting for scroll coordinates to update, a=testonly

Comment 4

[Narcis Beleuzu [:NarcisB]]

https://hg.mozilla.org/mozilla-central/rev/e184c11621a6