Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: eb9b59aba2a61af81ab34e2862f8e3cd599fe2f6
      
https://github.com/WebKit/WebKit/commit/eb9b59aba2a61af81ab34e2862f8e3cd599fe2f6
  Author: Alex Christensen <achristen...@apple.com>
  Date:   2024-09-20 (Fri, 20 Sep 2024)

  Changed paths:
    M Source/WebKit/UIProcess/FrameProcess.cpp
    M Source/WebKit/UIProcess/WebProcessProxy.cpp
    M Source/WebKit/UIProcess/WebProcessProxy.h

  Log Message:
  -----------
  Fix PLT with site isolation enabled
https://bugs.webkit.org/show_bug.cgi?id=280100
rdar://136397294

Reviewed by Ryosuke Niwa.

PLT has a site in it that is loaded in the main frame and then loaded in
an iframe under a different site's main frame.  Because the WebProcessPool's
process cache is currently designed and optimized for PSON, we can't use
the process cache the same way with site isolation when looking for a
process for an iframe because otherwise the BrowsingContextGroup's process
accounting gets messed up and we end up calling WebPage::loadRequest
with a frame identifier that doesn't exist in that process and it returns
early then the test hangs.  At some point we will need to teach the
WebProcessPool's process cache about site isolation to get perf up, but
until then we need to run the test to completion.

* Source/WebKit/UIProcess/WebProcessPool.cpp:
(WebKit::WebProcessPool::processForRegistrableDomain):

Canonical link: https://commits.webkit.org/284029@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

Reply via email to