Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 0d30af72a1b23a3fbe4c9b2a681b609c75dd7a02
      
https://github.com/WebKit/WebKit/commit/0d30af72a1b23a3fbe4c9b2a681b609c75dd7a02
  Author: Sihui Liu <sihui_...@apple.com>
  Date:   2024-04-22 (Mon, 22 Apr 2024)

  Changed paths:
    M Source/WebKit/UIProcess/WebFrameProxy.cpp
    M Source/WebKit/UIProcess/WebPageProxy.cpp
    M Source/WebKit/UIProcess/WebProcessPool.cpp

  Log Message:
  -----------
  Avoid duplicate NetworkProcess::AddAllowedFirstPartyForCookies messages
https://bugs.webkit.org/show_bug.cgi?id=272731
rdar://126531726

Reviewed by Alex Christensen.

Network process currently receives multiple 
NetworkProcess::AddAllowedFirstPartyForCookies messages for the same process
identifier and site pair, even though 
NetworkProcessProxy::addAllowedFirstPartyForCookies asks to not send the message
when the entry already exists. This is caused by some call sites explicitly 
sending the AddAllowedFirstPartyForCookies
message to network process instead of invoking 
NetworkProcessProxy::addAllowedFirstPartyForCookies. This patch fixes
that issue.

This patch also fixes a use-after-move issue in 
WebFrameProxy::prepareForProvisionalNavigationInProcess.

* Source/WebKit/UIProcess/WebFrameProxy.cpp:
(WebKit::WebFrameProxy::prepareForProvisionalNavigationInProcess):
* Source/WebKit/UIProcess/WebPageProxy.cpp:
(WebKit::WebPageProxy::loadAlternateHTML):
(WebKit::WebPageProxy::continueNavigationInNewProcess):
(WebKit::WebPageProxy::triggerBrowsingContextGroupSwitchForNavigation):
* Source/WebKit/UIProcess/WebProcessPool.cpp:
(WebKit::WebProcessPool::establishRemoteWorkerContextConnectionToNetworkProcess):

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