Branch: refs/heads/main
Home: https://github.com/WebKit/WebKit
Commit: 9680c16b5ad6b77fb485f5f7d2a4c04cb5e12712
https://github.com/WebKit/WebKit/commit/9680c16b5ad6b77fb485f5f7d2a4c04cb5e12712
Author: Cole Carley <[email protected]>
Date: 2026-05-04 (Mon, 04 May 2026)
Changed paths:
M LayoutTests/js/dom/modules/import-simple.html
M LayoutTests/platform/ios-site-isolation/TestExpectations
M LayoutTests/platform/mac-site-isolation/TestExpectations
Log Message:
-----------
[Site Isolation] js/dom/modules/import-simple.html fails
https://bugs.webkit.org/show_bug.cgi?id=313996
rdar://176198976
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-simple.html:
* LayoutTests/platform/ios-site-isolation/TestExpectations:
* LayoutTests/platform/mac-site-isolation/TestExpectations:
Canonical link: https://commits.webkit.org/312566@main
To unsubscribe from these emails, change your notification settings at
https://github.com/WebKit/WebKit/settings/notifications
