Branch: refs/heads/main
Home: https://github.com/WebKit/WebKit
Commit: c927c492d5b93193bc5cae300f1ca39eb36cd74c
https://github.com/WebKit/WebKit/commit/c927c492d5b93193bc5cae300f1ca39eb36cd74c
Author: Alex Christensen <achristen...@apple.com>
Date: 2024-04-08 (Mon, 08 Apr 2024)
Changed paths:
M LayoutTests/platform/mac-site-isolation/TestExpectations
M Source/WebKit/UIProcess/BrowsingContextGroup.cpp
Log Message:
-----------
Allow BrowsingContextGroup to have multiple processes with empty domains
https://bugs.webkit.org/show_bug.cgi?id=272355
rdar://126097189
Reviewed by Charlie Wolfe.
If a process is prewarmed and doesn't yet have a domain assigned, or if we load
a URL like a data URL in an iframe, we get a FrameProcess with an empty domain.
If this happens multiple times, we would hit an assertion that we don't have a
process in the map for the given domain. For empty domains, this assertion was
incorrect, and it was hit by the layout test
imported/w3c/web-platform-tests/html/browsers/history/the-location-interface/location-protocol-setter-non-broken.html
which no longer asserts but it does time out when run in a cross-origin iframe
with or without site isolation enabled.
* LayoutTests/platform/mac-site-isolation/TestExpectations:
* Source/WebKit/UIProcess/BrowsingContextGroup.cpp:
(WebKit::BrowsingContextGroup::ensureProcessForDomain):
(WebKit::BrowsingContextGroup::addFrameProcess):
Canonical link: https://commits.webkit.org/277220@main
To unsubscribe from these emails, change your notification settings at
https://github.com/WebKit/WebKit/settings/notifications
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes
