Bug 235131

Summary: css/css-transitions/before-load-001.html is a unique failure
Product: WebKit Reporter: Antoine Quint <graouts>
Component: AnimationsAssignee: Antoine Quint <graouts>
Status: RESOLVED FIXED    
Severity: Normal CC: achristensen, cdumez, dino, graouts, rniwa, webkit-bug-importer, youennf
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
See Also: https://bugs.webkit.org/show_bug.cgi?id=203416
https://bugs.webkit.org/show_bug.cgi?id=241676
Bug Depends on:    
Bug Blocks: 235130    

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
<rdar://problem/87785218>
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.