Branch: refs/heads/main Home: https://github.com/WebKit/WebKit Commit: 5959edcd8d6d5c936aa40e17626a25ddbfc09b51 https://github.com/WebKit/WebKit/commit/5959edcd8d6d5c936aa40e17626a25ddbfc09b51 Author: Alex Christensen <achristen...@apple.com> Date: 2025-03-31 (Mon, 31 Mar 2025)
Changed paths: M Source/WebKit/UIProcess/Notifications/ServiceWorkerNotificationHandler.cpp M Source/WebKit/UIProcess/WebGeolocationManagerProxy.cpp M Source/WebKit/UIProcess/WebPageProxy.cpp M Source/WebKit/UIProcess/WebPageProxy.h M Source/WebKit/UIProcess/WebProcessProxy.cpp M Source/WebKit/UIProcess/WebProcessProxy.h Log Message: ----------- Reduce use of WebPageProxy::m_legacyMainFrameProcess https://bugs.webkit.org/show_bug.cgi?id=290792 rdar://148272706 Reviewed by Brady Eidson. When IPC is received through a Connection to a receiver that only receives messages from a WebProcessProxy, we can safely assume that the result of what used to be WebProcessProxy::processForConnection is non-null and the same process that sent the message. I renamed it to WebProcessProxy::fromConnection and made it return a non-nullable Ref<WebProcessProxy>. This gives us a tool to help replace a lot of the use of m_legacyMainFrameProcess with the WebProcessProxy that sent the message. With site isolation off, there is no change in behavior. With site isolation on, this makes many things more correct. Also, to get to the process pool we can go through the configuration rather than the main frame process. Same result either way. * Source/WebKit/UIProcess/Notifications/ServiceWorkerNotificationHandler.cpp: (WebKit::ServiceWorkerNotificationHandler::sharedPreferencesForWebProcess const): * Source/WebKit/UIProcess/WebGeolocationManagerProxy.cpp: (WebKit::WebGeolocationManagerProxy::sharedPreferencesForWebProcess const): (WebKit::WebGeolocationManagerProxy::startUpdating): (WebKit::WebGeolocationManagerProxy::stopUpdating): (WebKit::WebGeolocationManagerProxy::setEnableHighAccuracy): * Source/WebKit/UIProcess/WebPageProxy.cpp: (WebKit::WebPageProxy::sharedPreferencesForWebProcess const): (WebKit::WebPageProxy::updateHiddenPageThrottlingAutoIncreases): (WebKit::WebPageProxy::receivedPolicyDecision): (WebKit::WebPageProxy::receivedNavigationResponsePolicyDecision): (WebKit::WebPageProxy::updateScrollingMode): (WebKit::WebPageProxy::updateSandboxFlags): (WebKit::WebPageProxy::didFailProvisionalLoadForFrame): (WebKit::WebPageProxy::didCommitLoadForFrame): (WebKit::WebPageProxy::didFinishDocumentLoadForFrame): (WebKit::WebPageProxy::didFinishLoadForFrame): (WebKit::WebPageProxy::didFailLoadForFrame): (WebKit::WebPageProxy::didSameDocumentNavigationForFrame): (WebKit::WebPageProxy::didSameDocumentNavigationForFrameViaJS): (WebKit::WebPageProxy::didChangeMainDocument): (WebKit::WebPageProxy::didFirstVisuallyNonEmptyLayoutForFrame): (WebKit::WebPageProxy::didDisplayInsecureContentForFrame): (WebKit::WebPageProxy::didRunInsecureContentForFrame): (WebKit::WebPageProxy::decidePolicyForNavigationActionAsync): (WebKit::WebPageProxy::decidePolicyForNewWindowAction): (WebKit::WebPageProxy::decidePolicyForResponse): (WebKit::WebPageProxy::decidePolicyForResponseShared): (WebKit::WebPageProxy::triggerBrowsingContextGroupSwitchForNavigation): (WebKit::WebPageProxy::willSubmitForm): (WebKit::WebPageProxy::didNavigateWithNavigationData): (WebKit::WebPageProxy::didPerformClientRedirect): (WebKit::WebPageProxy::didPerformServerRedirect): (WebKit::WebPageProxy::backForwardAddItemShared): (WebKit::WebPageProxy::contextMenuItemSelected): (WebKit::WebPageProxy::showNotification): (WebKit::WebPageProxy::cancelNotification): (WebKit::WebPageProxy::clearNotifications): (WebKit::WebPageProxy::didDestroyNotification): (WebKit::WebPageProxy::startURLSchemeTask): * Source/WebKit/UIProcess/WebPageProxy.h: * Source/WebKit/UIProcess/WebProcessProxy.cpp: (WebKit::WebProcessProxy::fromConnection): (WebKit::WebProcessProxy::processForConnection): Deleted. * Source/WebKit/UIProcess/WebProcessProxy.h: Canonical link: https://commits.webkit.org/292971@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