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