Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 258b6e614b2af8820d6adacccf3b5f0eb89ed33a
      
https://github.com/WebKit/WebKit/commit/258b6e614b2af8820d6adacccf3b5f0eb89ed33a
  Author: Alex Christensen <[email protected]>
  Date:   2026-09-09 (Wed, 09 Sep 2026)

  Changed paths:
    M Source/WebKit/UIProcess/BrowsingContextGroup.cpp
    M Source/WebKit/UIProcess/WebPageProxy.cpp
    M Tools/TestWebKitAPI/Tests/WebKit/WKWebView/EnhancedSecurityPolicies.mm

  Log Message:
  -----------
  Fix assertion and enable enhanced security with site isolation enabled
https://bugs.webkit.org/show_bug.cgi?id=323459
rdar://184250733

Reviewed by Sihui Liu.

This assertion was failing in 
BrowsingContextGroup::addFrameProcessWithoutInjectingPageContext:
!m_processMap.get(site) || canReplaceFrameProcessInProcessMap(site, 
*m_processMap.get(site))
The assertion just needed a slight update to allow replacing processes with 
different
enhanced
security states.

All the tests passed already, this just makes them also pass in debug builds 
with site isolation.

Test: Tools/TestWebKitAPI/Tests/WebKit/WKWebView/EnhancedSecurityPolicies.mm

* Source/WebKit/UIProcess/BrowsingContextGroup.cpp:
(WebKit::canReplaceFrameProcessInProcessMap):
(WebKit::BrowsingContextGroup::addFrameProcessWithoutInjectingPageContext):
(WebKit::BrowsingContextGroup::removeFrameProcess):
* Source/WebKit/UIProcess/WebPageProxy.cpp:
(WebKit::shouldUseEnhancedSecurityHeuristics):
(WebKit::canMoveSiteToEnhancedSecurityProcess):
(WebKit::WebPageProxy::triggerProcessSwapForEnhancedSecurity):
* Tools/TestWebKitAPI/Tests/WebKit/WKWebView/EnhancedSecurityPolicies.mm:
(runIframeKeepsSiteOutOfEnhancedSecurityProcess):

Canonical link: 
https://flagged.apple.com:443/proxy?t2=DZ8K5X0ar4&o=aHR0cHM6Ly9jb21taXRzLndlYmtpdC5vcmcvMzIwNzgxQG1haW4=&emid=8d0cf34f-ca9f-4521-81ad-26ce24e79c2f&c=11



To unsubscribe from these emails, change your
notification settings at https://github.com/WebKit/WebKit/settings/notifications

Reply via email to