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