RESOLVED FIXED245680
REGRESSION?(254483@main): [ iOS16 macOS wk2 ] http/tests/misc/iframe-shadow-realm.html is constantly timing out
https://bugs.webkit.org/show_bug.cgi?id=245680
Summary REGRESSION?(254483@main): [ iOS16 macOS wk2 ] http/tests/misc/iframe-shadow-r...
Hercules Hjalmarsson
Reported 2022-09-26 09:30:20 PDT
http/tests/misc/iframe-shadow-realm.html Is a constant timeout on iOS16 and macOS wk2 ToT and since 254483@main. It appears that it was expected to fail after 254483@main. However, it wasn't expected to timeout judging by the expectations set. HISTORY: https://results.webkit.org/?suite=layout-tests&test=http/tests/misc/iframe-shadow-realm.html DIFF: @@ -1,10 +1,9 @@ +CONSOLE MESSAGE: Unhandled Promise Rejection: ReferenceError: Can't find variable: ShadowRealm +FAIL: Timed out waiting for notifyDone to be called + Test that shadow realms can load modules from different-origin iframe contexts On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE". -PASS event.data is 42 -PASS successfullyParsed is true -TEST COMPLETE -
Attachments
Radar WebKit Bug Importer
Comment 1 2022-09-26 09:30:49 PDT
Hercules Hjalmarsson
Comment 2 2022-09-26 09:31:48 PDT
This issue can be bisected to 254483@main using command: run-webkit-tests --iterations=2 --ios-simulator http/tests/misc/iframe-shadow-realm.html
EWS
Comment 3 2022-09-26 10:46:48 PDT
Test gardening commit 254869@main (a7f739402edf): <https://commits.webkit.org/254869@main> Reviewed commits have been landed. Closing PR #4713 and removing active labels.
Mark Lam
Comment 4 2022-09-27 09:35:46 PDT
This is probably just fall out for disabling ShadowRealm. All we need to do for this test is add this at the top of the file: <!-- webkit-test-runner [ jscOptions=--useShadowRealm=true ] -->
Mark Lam
Comment 5 2022-09-28 08:18:33 PDT
Yusuke tells me that this test as currently written can timeout. So, this is expected behavior.
EWS
Comment 6 2022-09-28 08:38:54 PDT
Test gardening commit 254957@main (a53a5529e983): <https://commits.webkit.org/254957@main> Reviewed commits have been landed. Closing PR #4795 and removing active labels.
Mark Lam
Comment 7 2022-09-28 15:41:42 PDT
FYI, skipping the test is the right thing to do here (which Hercules just landed a patch for) because a test expectation of [Pass Fail Timeout] gives us no information, and wastes EWS time with time outs.
Note You need to log in before you can comment on or make changes to this bug.