Branch: refs/heads/main
Home: https://github.com/WebKit/WebKit
Commit: 7afa906f4a5e837f4c9ef9ca8ae57f653e80c91e
https://github.com/WebKit/WebKit/commit/7afa906f4a5e837f4c9ef9ca8ae57f653e80c91e
Author: Cole Carley <[email protected]>
Date: 2026-05-05 (Tue, 05 May 2026)
Changed paths:
M LayoutTests/js/dom/modules/import-from-javascript-url.html
M LayoutTests/platform/ios-site-isolation/TestExpectations
M LayoutTests/platform/mac-site-isolation/TestExpectations
Log Message:
-----------
[Site Isolation] js/dom/modules/import-from-javascript-url.html fails
https://bugs.webkit.org/show_bug.cgi?id=314001
rdar://176201148
Reviewed by Megan Gardner and Ryosuke Niwa.
The test is showing additional lines that say "TEST COMPLETE"
and "PASS successfullyParsed is true" from the file that is dynamically
imported. That imported file calls finishJSTest, then in the importing
file, finshJSTest is called again.
Without site isolation, the first call to finishJSTest reports it's
messages and ends the test before the second call to finishJSTest.
In the site isolation case, the additional IPC overhead allows for both
functions to report their messages.
To fix this issue, I added a 0s delay before calling the second
finishJSTest, which ensures for a graceful exit from the first call to
finishJSTest.
* LayoutTests/js/dom/modules/import-from-javascript-url.html:
* LayoutTests/platform/ios-site-isolation/TestExpectations:
* LayoutTests/platform/mac-site-isolation/TestExpectations:
Canonical link: https://commits.webkit.org/312671@main
To unsubscribe from these emails, change your notification settings at
https://github.com/WebKit/WebKit/settings/notifications
