Closed Bug 1626324 Opened 4 years ago Closed 4 years ago

[wpt-sync] Sync PR 22573 - De-flake two lazyload WPT tests.

Categories

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

task

Tracking

()

RESOLVED FIXED
mozilla77
Tracking Status
firefox77 --- fixed

People

(Reporter: mozilla.org, Unassigned)

References

()

Details

(Whiteboard: [wptsync downstream])

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

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

Chris Harrelson <chrishtr@chromium.org> wrote:

De-flake two lazyload WPT tests.

They should only set the src of an img after registering the onload
handler, to avoid the onload handler missing the load event.

See also discussion here about the flakiness this is fixing:
https://github.com/web-platform-tests/wpt/pull/22375

Change-Id: I78707beb1cd2ce609d948dcc1e5b22a6e3b1b58b
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2127524
Reviewed-by: Stefan Zager \<szager@chromium.org>
Commit-Queue: Chris Harrelson \<chrishtr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#754577}

PR 22573 applied with additional changes from upstream: fa480cec078652d482767d1e3daf5bfd7444dbea
Component: web-platform-tests → DOM: Core & HTML
Product: Testing → Core
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → INVALID
Status: RESOLVED → REOPENED
Resolution: INVALID → ---
Pushed by wptsync@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/4c47235228b0
[wpt PR 22573] - De-flake two lazyload WPT tests., a=testonly
Test result changes from PR not available.
Status: REOPENED → RESOLVED
Closed: 4 years ago4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla77
You need to log in before you can comment on or make changes to this bug.