[webkit-changes] [WebKit/WebKit] 48aceb: Rename PolicyAction::StopAllLoads to LoadWillConti...

Alex Christensen noreply at github.com
Tue Aug 1 10:50:30 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 48aceba044ae0a79621044df8c83d47232fa9491
      https://github.com/WebKit/WebKit/commit/48aceba044ae0a79621044df8c83d47232fa9491
  Author: Alex Christensen <achristensen at apple.com>
  Date:   2023-08-01 (Tue, 01 Aug 2023)

  Changed paths:
    M Source/WebCore/loader/DocumentLoader.cpp
    M Source/WebCore/loader/FrameLoader.cpp
    M Source/WebCore/loader/FrameLoaderTypes.h
    M Source/WebCore/loader/PolicyChecker.cpp
    M Source/WebCore/loader/PolicyChecker.h
    M Source/WebKit/NetworkProcess/NetworkDataTaskBlob.cpp
    M Source/WebKit/NetworkProcess/NetworkDataTaskDataURL.cpp
    M Source/WebKit/NetworkProcess/cocoa/NetworkSessionCocoa.mm
    M Source/WebKit/NetworkProcess/soup/NetworkDataTaskSoup.cpp
    M Source/WebKit/Shared/WebCoreArgumentCoders.serialization.in
    M Source/WebKit/UIProcess/WebPageProxy.cpp

  Log Message:
  -----------
  Rename PolicyAction::StopAllLoads to LoadWillContinueInAnotherProcess
https://bugs.webkit.org/show_bug.cgi?id=259671
rdar://113171936

Reviewed by Brady Eidson.

It was initially used for process swapping on navigation where we told the old process to stop
basically everything it was doing.  With site isolation, we are using it when loading iframes,
and I had to do a little logic change to only have the iframe stop what it was doing, but not
prepare for the process to suspend.  The meaning of this enum value means most consistently in
all its current use that the load is continuing but in another process.

* Source/WebCore/loader/DocumentLoader.cpp:
(WebCore::DocumentLoader::willSendRequest):
(WebCore::DocumentLoader::continueAfterContentPolicy):
* Source/WebCore/loader/FrameLoader.cpp:
(WebCore::FrameLoader::continueLoadAfterNavigationPolicy):
* Source/WebCore/loader/FrameLoaderTypes.h:
* Source/WebCore/loader/PolicyChecker.cpp:
(WebCore::FrameLoader::PolicyChecker::checkNavigationPolicy):
(WebCore::FrameLoader::PolicyChecker::checkNewWindowPolicy):
* Source/WebCore/loader/PolicyChecker.h:
* Source/WebKit/NetworkProcess/NetworkDataTaskBlob.cpp:
(WebKit::NetworkDataTaskBlob::dispatchDidReceiveResponse):
* Source/WebKit/NetworkProcess/cocoa/NetworkSessionCocoa.mm:
(toNSURLSessionResponseDisposition):
* Source/WebKit/Shared/WebCoreArgumentCoders.serialization.in:
* Source/WebKit/UIProcess/WebPageProxy.cpp:
(WebKit::WebPageProxy::receivedNavigationPolicyDecision):

Canonical link: https://commits.webkit.org/266484@main




More information about the webkit-changes mailing list