Bug 254528 - New test: [macOS WK1] imported/w3c/web-platform-tests/preload/onerror-event.html is a flaky failure
Summary: New test: [macOS WK1] imported/w3c/web-platform-tests/preload/onerror-event.h...
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: Other
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Ryan Haddad
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2023-03-27 09:12 PDT by Ryan Haddad
Modified: 2023-03-27 09:21 PDT (History)
3 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Ryan Haddad 2023-03-27 09:12:37 PDT
imported/w3c/web-platform-tests/preload/onerror-event.html has been a flaky failure on macOS WK1 bots since it was added with https://commits.webkit.org/260462@main.

It appears that a console message is coming in out of the expected order.

--- /Volumes/Data/worker/Apple-Monterey-Release-WK1-Tests/build/layout-test-results/imported/w3c/web-platform-tests/preload/onerror-event-expected.txt
+++ /Volumes/Data/worker/Apple-Monterey-Release-WK1-Tests/build/layout-test-results/imported/w3c/web-platform-tests/preload/onerror-event-actual.txt
@@ -2,12 +2,12 @@
 CONSOLE MESSAGE: <link rel=preload> cannot have the empty string as `as` value
 CONSOLE MESSAGE: The resource http://localhost:8800/preload/non-existent/dummy.js was preloaded using link preload but not used within a few seconds from the window's load event. Please make sure it wasn't preloaded for nothing.
 CONSOLE MESSAGE: The resource http://localhost:8800/preload/non-existent/dummy.css was preloaded using link preload but not used within a few seconds from the window's load event. Please make sure it wasn't preloaded for nothing.
-CONSOLE MESSAGE: The resource http://localhost:8800/preload/non-existent/square.png was preloaded using link preload but not used within a few seconds from the window's load event. Please make sure it wasn't preloaded for nothing.
 CONSOLE MESSAGE: The resource http://localhost:8800/preload/non-existent/Ahem.ttf was preloaded using link preload but not used within a few seconds from the window's load event. Please make sure it wasn't preloaded for nothing.
 CONSOLE MESSAGE: The resource http://localhost:8800/preload/non-existent/test.mp4 was preloaded using link preload but not used within a few seconds from the window's load event. Please make sure it wasn't preloaded for nothing.
 CONSOLE MESSAGE: The resource http://localhost:8800/preload/non-existent/test.oga was preloaded using link preload but not used within a few seconds from the window's load event. Please make sure it wasn't preloaded for nothing.
 CONSOLE MESSAGE: The resource http://localhost:8800/preload/non-existent/security/captions.vtt was preloaded using link preload but not used within a few seconds from the window's load event. Please make sure it wasn't preloaded for nothing.
 CONSOLE MESSAGE: The resource http://localhost:8800/preload/non-existent/dummy.xml?fetch was preloaded using link preload but not used within a few seconds from the window's load event. Please make sure it wasn't preloaded for nothing.
+CONSOLE MESSAGE: The resource http://localhost:8800/preload/non-existent/square.png was preloaded using link preload but not used within a few seconds from the window's load event. Please make sure it wasn't preloaded for nothing.
 CONSOLE MESSAGE: Error: assert_true: video triggered error event expected true got false
 
 FAIL Makes sure that preloaded resources trigger the onerror event Error: assert_true: video triggered error event expected true got false

https://results.webkit.org/?suite=layout-tests&test=imported%2Fw3c%2Fweb-platform-tests%2Fpreload%2Fonerror-event.html
Comment 1 Radar WebKit Bug Importer 2023-03-27 09:12:49 PDT
<rdar://problem/107269834>
Comment 2 Ryan Haddad 2023-03-27 09:16:11 PDT
Pull request: https://github.com/WebKit/WebKit/pull/12009
Comment 3 EWS 2023-03-27 09:21:33 PDT
Test gardening commit 262153@main (4464ceb97d52): <https://commits.webkit.org/262153@main>

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