245680 2022-09-26 09:30:20 -0700 REGRESSION?(254483@main): [ iOS16 macOS wk2 ] http/tests/misc/iframe-shadow-realm.html is constantly timing out 2022-09-28 15:41:42 -0700 1 1 1 Unclassified WebKit New Bugs WebKit Nightly Build Unspecified Unspecified RESOLVED FIXED InRadar P2 Normal --- 1 hhjalmarsson webkit-unassigned mark.lam webkit-bot-watchers-bugzilla webkit-bug-importer ysuzuki oldest_to_newest 1901206 0 hhjalmarsson 2022-09-26 09:30:20 -0700 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 - 1901208 1 webkit-bug-importer 2022-09-26 09:30:49 -0700 <rdar://problem/100414380> 1901210 2 hhjalmarsson 2022-09-26 09:31:48 -0700 This issue can be bisected to 254483@main using command: run-webkit-tests --iterations=2 --ios-simulator http/tests/misc/iframe-shadow-realm.html 1901225 3 ews-feeder 2022-09-26 10:46:48 -0700 Test gardening commit 254869@main (a7f739402edf): <https://commits.webkit.org/254869@main> Reviewed commits have been landed. Closing PR #4713 and removing active labels. 1901552 4 mark.lam 2022-09-27 09:35:46 -0700 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 ] --> 1901784 5 mark.lam 2022-09-28 08:18:33 -0700 Yusuke tells me that this test as currently written can timeout. So, this is expected behavior. 1901792 6 ews-feeder 2022-09-28 08:38:54 -0700 Test gardening commit 254957@main (a53a5529e983): <https://commits.webkit.org/254957@main> Reviewed commits have been landed. Closing PR #4795 and removing active labels. 1901889 7 mark.lam 2022-09-28 15:41:42 -0700 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.