1774545 - [wpt-sync] Sync PR 34463 - Fix Safari harness timeout in css/css-transitions/before-load-001.html

Status: RESOLVED FIXED

(Core :: CSS Transitions and Animations, task, P4)

Description

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

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

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

Antoine Quint <[e-mail]> wrote:

Fix Safari harness timeout in css/css-transitions/before-load-001.html

The test css/css-transitions/before-load-001.html relies on loading an image where the load is set to 100 seconds such that the window load event dispatch is delayed until a CSS Transition is ran. At that point, the image's source is set to the empty string to stop the image load and have the window load event dispatched.

Safari has an issue where an <img> pointing to a long-loading resource will continue delaying dispatch of the load event on the window even when the image source has been reset. This issue is tracked by https://bugs.webkit.org/show_bug.cgi?id=241676.

As a result, css/css-transitions/before-load-001.html causes a harness timeout in Safari even though the ability to run a CSS Transition before the window load event has been dispatched works correctly and the test assertions pass.

We work around that issue by lowering the load duration for the image to 1 second instead of 100. This doesn't changed the functionality that this test is explicitly testing. And to explicitly test that resetting a long-loading image source no longer delays the dispatch of the window load event, a new dedicated test is added, which will itself cause a timeout in Safari, but that is what is expected.

Addresses https://github.com/web-platform-tests/wpt/issues/34444.

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=d2a166e7576ba5ae3ff08a6f032042fce38f747b

Comment 2

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

CI Results

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

Total 2 tests and 1 subtests

Status Summary

Firefox

OK : 2
PASS : 2

Chrome

OK : 2
PASS : 2

Safari

OK : 1
PASS : 1
TIMEOUT: 2

Links

GitHub PR Head
GitHub PR Base

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/5e8eb0e4b12a
[wpt PR 34463] - Fix Safari harness timeout in css/css-transitions/before-load-001.html, a=testonly

Comment 4

[Cristian Tuns]

https://hg.mozilla.org/mozilla-central/rev/5e8eb0e4b12a