Bug 235131 - css/css-transitions/before-load-001.html is a unique failure
Summary: css/css-transitions/before-load-001.html is a unique failure
Alias: None
Product: WebKit
Classification: Unclassified
Component: Animations (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Antoine Quint
Keywords: InRadar
Depends on:
Blocks: 235130
  Show dependency treegraph
Reported: 2022-01-12 12:12 PST by Antoine Quint
Modified: 2022-06-17 04:57 PDT (History)
7 users (show)

See Also:


Note You need to log in before you can comment on or make changes to this bug.
Description Antoine Quint 2022-01-12 12:12:29 PST
WebKit has some unique failures for css/css-transitions/before-load-001.html. See https://wpt.fyi/results/css/css-transitions/before-load-001.html?label=experimental&label=master&aligned.
Comment 1 Radar WebKit Bug Importer 2022-01-19 12:23:33 PST
Comment 2 Antoine Quint 2022-06-14 06:06:17 PDT
Odd, the test assertion passes but there is a timeout harness error!
Comment 3 Antoine Quint 2022-06-15 04:23:18 PDT
I believe the delay set on the <img> in that test correctly delays the `load` event on the page by 100 seconds which causes the timeout.

I filed https://github.com/web-platform-tests/wpt/issues/34444 to discuss a change to the test.
Comment 4 Antoine Quint 2022-06-15 07:53:57 PDT
No, the issue is that when we interrupt the <img> load the window load event isn't fired and rather still waits on that resource being loaded. So this test is really a loading bug.
Comment 5 Antoine Quint 2022-06-16 01:33:23 PDT
I filed https://github.com/web-platform-tests/wpt/pull/34463 to modify the test and created one that specifically test for the window `load` delay and canceling long image loads.

I filed https://bugs.webkit.org/show_bug.cgi?id=241676 to deal with that issue.
Comment 6 Antoine Quint 2022-06-16 11:58:16 PDT
Pull request: https://github.com/WebKit/WebKit/pull/1584
Comment 7 EWS 2022-06-17 04:57:50 PDT
Committed r295632 (251637@main): <https://commits.webkit.org/251637@main>

Reviewed commits have been landed. Closing PR #1584 and removing active labels.