Closed Bug 1650908 Opened 4 years ago Closed 4 years ago

[wpt-sync] Sync PR 24471 - Change 'window.load() event' to 'window load event' for correctness

Categories

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

task

Tracking

()

RESOLVED FIXED
mozilla80
Tracking Status
firefox80 --- fixed

People

(Reporter: mozilla.org, Unassigned)

References

(Blocks 1 open bug, )

Details

(Whiteboard: [wptsync downstream])

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

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

Simon Pieters <zcorpan@gmail.com> wrote:

Change 'window.load() event' to 'window load event' for correctness

I noticed this while reviewing https://github.com/web-platform-tests/wpt/pull/24361 (cc @domfarolino @scott-little )

Component: web-platform-tests → DOM: Core & HTML
Product: Testing → Core

CI Results

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

Total 4 tests and 1 subtests

Status Summary

Firefox

OK : 4
PASS: 3
FAIL: 4

Chrome

OK : 4
PASS: 6
FAIL: 1

Safari

OK : 4
PASS: 3
FAIL: 4

Links

Gecko CI (Treeherder)
GitHub PR Head
GitHub PR Base

Details

New Tests That Don't Pass

/html/semantics/embedded-content/the-img-element/image-loading-lazy-in-viewport-dynamic.html
Test that in viewport images with loading='lazy' and changed to loading='eager' do not block the window load event.: FAIL (Chrome: PASS, Safari: FAIL)
/html/semantics/embedded-content/the-iframe-element/iframe-loading-lazy.tentative.html
Below-viewport srcdoc iframes load lazily: FAIL (Chrome: FAIL, Safari: FAIL)
Below-viewport iframes load lazily: FAIL (Chrome: PASS, Safari: FAIL)
/html/semantics/embedded-content/the-img-element/image-loading-lazy-below-viewport-dynamic.html
Test that below viewport images with loading='lazy' and changed to loading='eager' load and do not block the window load event.: FAIL (Chrome: PASS, Safari: FAIL)

Pushed by wptsync@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/d2c32f3b331f
[wpt PR 24471] - Change 'window.load() event' to 'window load event' for correctness, a=testonly
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla80
You need to log in before you can comment on or make changes to this bug.