Branch: refs/heads/main Home: https://github.com/WebKit/WebKit Commit: c5175b3ba737b9c4d0237e82c887d049bd24f9a1 https://github.com/WebKit/WebKit/commit/c5175b3ba737b9c4d0237e82c887d049bd24f9a1 Author: Ryosuke Niwa <rn...@webkit.org> Date: 2024-08-16 (Fri, 16 Aug 2024)
Changed paths: M Source/WebKit/GPUProcess/GPUConnectionToWebProcess.cpp M Source/WebKit/GPUProcess/graphics/WebGPU/RemoteCommandEncoder.cpp M Source/WebKit/GPUProcess/graphics/WebGPU/RemoteCompositorIntegration.cpp M Source/WebKit/GPUProcess/graphics/WebGPU/RemoteDevice.cpp M Source/WebKit/GPUProcess/graphics/WebGPU/RemoteGPU.cpp M Source/WebKit/GPUProcess/graphics/WebGPU/RemoteRenderBundleEncoder.cpp M Source/WebKit/GPUProcess/graphics/WebGPU/RemoteTexture.cpp M Source/WebKit/GPUProcess/media/RemoteAudioSessionProxy.cpp M Source/WebKit/GPUProcess/media/RemoteSourceBufferProxy.cpp M Source/WebKit/GPUProcess/webrtc/RemoteMediaRecorder.cpp M Source/WebKit/ModelProcess/ModelConnectionToWebProcess.cpp M Source/WebKit/NetworkProcess/NetworkBroadcastChannelRegistry.cpp M Source/WebKit/NetworkProcess/NetworkConnectionToWebProcess.cpp M Source/WebKit/NetworkProcess/ServiceWorker/WebSWServerConnection.cpp M Source/WebKit/NetworkProcess/ServiceWorker/WebSWServerToContextConnection.cpp M Source/WebKit/NetworkProcess/SharedWorker/WebSharedWorkerServerConnection.cpp M Source/WebKit/NetworkProcess/storage/NetworkStorageManager.cpp M Source/WebKit/Platform/IPC/Connection.h M Source/WebKit/Shared/ApplePay/WebPaymentCoordinatorProxy.cpp M Source/WebKit/UIProcess/API/Cocoa/WKWebView.mm M Source/WebKit/UIProcess/AuxiliaryProcessProxy.h M Source/WebKit/UIProcess/Cocoa/GPUProcessProxyCocoa.mm M Source/WebKit/UIProcess/Cocoa/UserMediaCaptureManagerProxy.cpp M Source/WebKit/UIProcess/Cocoa/WebPageProxyCocoa.mm M Source/WebKit/UIProcess/Cocoa/WebPasteboardProxyCocoa.mm M Source/WebKit/UIProcess/Cocoa/WebProcessProxyCocoa.mm M Source/WebKit/UIProcess/Cocoa/XPCConnectionTerminationWatchdog.mm M Source/WebKit/UIProcess/CoordinatedGraphics/DrawingAreaProxyCoordinatedGraphics.cpp M Source/WebKit/UIProcess/Downloads/DownloadProxy.cpp M Source/WebKit/UIProcess/DrawingAreaProxy.cpp M Source/WebKit/UIProcess/GPU/GPUProcessProxy.cpp M Source/WebKit/UIProcess/Model/ModelProcessProxy.cpp M Source/WebKit/UIProcess/Network/NetworkProcessProxy.cpp M Source/WebKit/UIProcess/Network/NetworkProcessProxyCocoa.mm M Source/WebKit/UIProcess/ProvisionalPageProxy.cpp M Source/WebKit/UIProcess/RemoteLayerTree/RemoteLayerTreeDrawingAreaProxy.mm M Source/WebKit/UIProcess/RemoteLayerTree/RemoteScrollingCoordinatorProxy.cpp M Source/WebKit/UIProcess/RemoteLayerTree/mac/RemoteLayerTreeDrawingAreaProxyMac.mm M Source/WebKit/UIProcess/SpeechRecognitionRemoteRealtimeMediaSourceManager.cpp M Source/WebKit/UIProcess/SpeechRecognitionServer.cpp M Source/WebKit/UIProcess/WebPageProxy.cpp M Source/WebKit/UIProcess/WebPageProxyTesting.cpp M Source/WebKit/UIProcess/WebProcessProxy.cpp M Source/WebKit/UIProcess/WebsiteData/Cocoa/WebsiteDataStoreCocoa.mm M Source/WebKit/UIProcess/ios/WKContentView.mm M Source/WebKit/UIProcess/ios/WKContentViewInteraction.mm M Source/WebKit/UIProcess/ios/WebPageProxyIOS.mm M Source/WebKit/UIProcess/mac/SecItemShimProxy.cpp M Source/WebKit/UIProcess/mac/TiledCoreAnimationDrawingAreaProxy.mm M Source/WebKit/UIProcess/mac/WKImmediateActionController.mm M Source/WebKit/UIProcess/mac/WebPageProxyMac.mm Log Message: ----------- AuxiliaryProcessProxy::connection and MESSGE_CHECK_BASE should use a reference to IPC::Connection instead of a pointer https://bugs.webkit.org/show_bug.cgi?id=278202 Reviewed by Sihui Liu. This PR makes AuxiliaryProcessProxy::connection return a reference to IPC::Connection instead of a pointer since it already asserts that it's not null and most call site uses it without checking nullity, and MESSGE_CHECK_*BASE take a reference to IPC::Connection instead of a pointer. It also makes "connection" an explicit argument to various MESSAGE_CHECK macros for clarity. * Source/WebKit/GPUProcess/GPUConnectionToWebProcess.cpp: * Source/WebKit/GPUProcess/graphics/WebGPU/RemoteCommandEncoder.cpp: * Source/WebKit/GPUProcess/graphics/WebGPU/RemoteCompositorIntegration.cpp: * Source/WebKit/GPUProcess/graphics/WebGPU/RemoteDevice.cpp: * Source/WebKit/GPUProcess/graphics/WebGPU/RemoteGPU.cpp: * Source/WebKit/GPUProcess/graphics/WebGPU/RemoteRenderBundleEncoder.cpp: * Source/WebKit/GPUProcess/graphics/WebGPU/RemoteTexture.cpp: * Source/WebKit/GPUProcess/media/RemoteAudioSessionProxy.cpp: * Source/WebKit/GPUProcess/media/RemoteSourceBufferProxy.cpp: * Source/WebKit/GPUProcess/webrtc/RemoteMediaRecorder.cpp: * Source/WebKit/ModelProcess/ModelConnectionToWebProcess.cpp: * Source/WebKit/NetworkProcess/NetworkBroadcastChannelRegistry.cpp: (WebKit::NetworkBroadcastChannelRegistry::registerChannel): (WebKit::NetworkBroadcastChannelRegistry::unregisterChannel): (WebKit::NetworkBroadcastChannelRegistry::postMessage): * Source/WebKit/NetworkProcess/NetworkConnectionToWebProcess.cpp: * Source/WebKit/NetworkProcess/ServiceWorker/WebSWServerConnection.cpp: * Source/WebKit/NetworkProcess/ServiceWorker/WebSWServerToContextConnection.cpp: * Source/WebKit/NetworkProcess/SharedWorker/WebSharedWorkerServerConnection.cpp: * Source/WebKit/NetworkProcess/storage/NetworkStorageManager.cpp: (WebKit::NetworkStorageManager::connectToStorageArea): (WebKit::NetworkStorageManager::cancelConnectToStorageArea): (WebKit::NetworkStorageManager::disconnectFromStorageArea): (WebKit::NetworkStorageManager::setItem): (WebKit::NetworkStorageManager::removeItem): (WebKit::NetworkStorageManager::clear): * Source/WebKit/Platform/IPC/Connection.h: * Source/WebKit/Shared/ApplePay/WebPaymentCoordinatorProxy.cpp: * Source/WebKit/UIProcess/API/Cocoa/WKWebView.mm: (-[WKWebView _getContentsAsStringWithCompletionHandlerKeepIPCConnectionAliveForTesting:]): * Source/WebKit/UIProcess/AuxiliaryProcessProxy.h: (WebKit::AuxiliaryProcessProxy::connection const): (WebKit::AuxiliaryProcessProxy::protectedConnection const): (WebKit::AuxiliaryProcessProxy::sendSync): * Source/WebKit/UIProcess/Cocoa/UserMediaCaptureManagerProxy.cpp: * Source/WebKit/UIProcess/Cocoa/WebPageProxyCocoa.mm: (WebKit::WebPageProxy::saveRecentSearches): (WebKit::WebPageProxy::loadRecentSearches): (WebKit::WebPageProxy::createSandboxExtensionsIfNeeded): (WebKit::WebPageProxy::Internals::paymentCoordinatorConnection): (WebKit::WebPageProxy::addTextAnimationForAnimationID): (WebKit::WebPageProxy::removeTextAnimationForAnimationID): * Source/WebKit/UIProcess/Cocoa/WebPasteboardProxyCocoa.mm: (WebKit::WebPasteboardProxy::accessType const): (WebKit::WebPasteboardProxy::didModifyContentsOfPasteboard): (WebKit::WebPasteboardProxy::getPasteboardTypes): (WebKit::WebPasteboardProxy::getPasteboardPathnamesForType): (WebKit::WebPasteboardProxy::getPasteboardStringForType): (WebKit::WebPasteboardProxy::getPasteboardStringsForType): (WebKit::WebPasteboardProxy::getPasteboardBufferForType): (WebKit::WebPasteboardProxy::getPasteboardChangeCount): (WebKit::WebPasteboardProxy::getPasteboardColor): (WebKit::WebPasteboardProxy::getPasteboardURL): (WebKit::WebPasteboardProxy::addPasteboardTypes): (WebKit::WebPasteboardProxy::setPasteboardTypes): (WebKit::WebPasteboardProxy::setPasteboardURL): (WebKit::WebPasteboardProxy::setPasteboardColor): (WebKit::WebPasteboardProxy::setPasteboardStringForType): (WebKit::WebPasteboardProxy::containsURLStringSuitableForLoading): (WebKit::WebPasteboardProxy::urlStringSuitableForLoading): (WebKit::WebPasteboardProxy::setPasteboardBufferForType): (WebKit::WebPasteboardProxy::getNumberOfFiles): (WebKit::WebPasteboardProxy::typesSafeForDOMToReadAndWrite): (WebKit::WebPasteboardProxy::writeCustomData): (WebKit::WebPasteboardProxy::allPasteboardItemInfo): (WebKit::WebPasteboardProxy::informationForItemAtIndex): (WebKit::WebPasteboardProxy::getPasteboardItemsCount): (WebKit::WebPasteboardProxy::readStringFromPasteboard): (WebKit::WebPasteboardProxy::readURLFromPasteboard): (WebKit::WebPasteboardProxy::readBufferFromPasteboard): (WebKit::WebPasteboardProxy::containsStringSafeForDOMToReadForType): (WebKit::WebPasteboardProxy::writeURLToPasteboard): (WebKit::WebPasteboardProxy::writeWebContentToPasteboard): (WebKit::WebPasteboardProxy::writeImageToPasteboard): (WebKit::WebPasteboardProxy::writeStringToPasteboard): (WebKit::WebPasteboardProxy::determineDataOwner const): (WebKit::WebPasteboardProxy::testIPCSharedMemory): * Source/WebKit/UIProcess/Cocoa/WebProcessProxyCocoa.mm: (WebKit::WebProcessProxy::messageSourceIsValidWebContentProcess): (WebKit::WebProcessProxy::auditToken const): * Source/WebKit/UIProcess/Cocoa/XPCConnectionTerminationWatchdog.mm: (WebKit::XPCConnectionTerminationWatchdog::XPCConnectionTerminationWatchdog): * Source/WebKit/UIProcess/CoordinatedGraphics/DrawingAreaProxyCoordinatedGraphics.cpp: (WebKit::DrawingAreaProxyCoordinatedGraphics::forceUpdateIfNeeded): * Source/WebKit/UIProcess/Downloads/DownloadProxy.cpp: (WebKit::DownloadProxy::didReceiveAuthenticationChallenge): * Source/WebKit/UIProcess/DrawingAreaProxy.cpp: (WebKit::DrawingAreaProxy::messageSenderConnection const): * Source/WebKit/UIProcess/GPU/GPUProcessProxy.cpp: (WebKit::GPUProcessProxy::updateSandboxAccess): (WebKit::GPUProcessProxy::processWillShutDown): * Source/WebKit/UIProcess/Model/ModelProcessProxy.cpp: * Source/WebKit/UIProcess/Network/NetworkProcessProxy.cpp: (WebKit::NetworkProcessProxy::terminate): (WebKit::NetworkProcessProxy::processWillShutDown): (WebKit::NetworkProcessProxy::getNetworkProcessConnection): (WebKit::NetworkProcessProxy::didReceiveAuthenticationChallenge): (WebKit::NetworkProcessProxy::didExceedMemoryLimit): * Source/WebKit/UIProcess/Network/NetworkProcessProxyCocoa.mm: (WebKit::NetworkProcessProxy::sendXPCEndpointToProcess): * Source/WebKit/UIProcess/ProvisionalPageProxy.cpp: (WebKit::ProvisionalPageProxy::messageSenderConnection const): * Source/WebKit/UIProcess/RemoteLayerTree/RemoteLayerTreeDrawingAreaProxy.mm: (WebKit::RemoteLayerTreeDrawingAreaProxy::processStateForConnection): (WebKit::RemoteLayerTreeDrawingAreaProxy::didRefreshDisplay): (WebKit::RemoteLayerTreeDrawingAreaProxy::waitForDidUpdateActivityState): * Source/WebKit/UIProcess/RemoteLayerTree/RemoteScrollingCoordinatorProxy.cpp: * Source/WebKit/UIProcess/RemoteLayerTree/mac/RemoteLayerTreeDrawingAreaProxyMac.mm: (WebKit::RemoteLayerTreeDrawingAreaProxyMac::createFence): * Source/WebKit/UIProcess/SpeechRecognitionRemoteRealtimeMediaSourceManager.cpp: (WebKit::SpeechRecognitionRemoteRealtimeMediaSourceManager::messageSenderConnection const): * Source/WebKit/UIProcess/SpeechRecognitionServer.cpp: (WebKit::SpeechRecognitionServer::messageSenderConnection const): * Source/WebKit/UIProcess/WebPageProxy.cpp: (WebKit::WebPageProxy::handleMessage): (WebKit::WebPageProxy::handleSynchronousMessage): (WebKit::WebPageProxy::maybeInitializeSandboxExtensionHandle): (WebKit::WebPageProxy::createInspectorTarget): (WebKit::WebPageProxy::destroyInspectorTarget): (WebKit::WebPageProxy::executeEditCommand): (WebKit::WebPageProxy::sendWheelEvent): (WebKit::WebPageProxy::sendWheelEventScrollingAccelerationCurveIfNecessary): (WebKit::WebPageProxy::pageScaleFactorDidChange): (WebKit::WebPageProxy::pluginScaleFactorDidChange): (WebKit::WebPageProxy::pluginZoomFactorDidChange): (WebKit::WebPageProxy::didCreateMainFrame): (WebKit::WebPageProxy::didCreateSubframe): (WebKit::WebPageProxy::didDestroyFrame): (WebKit::WebPageProxy::didExplicitOpenForFrame): (WebKit::WebPageProxy::willPerformClientRedirectForFrame): (WebKit::WebPageProxy::didCancelClientRedirectForFrame): (WebKit::WebPageProxy::didFailProvisionalLoadForFrame): (WebKit::WebPageProxy::didCommitLoadForFrame): (WebKit::WebPageProxy::didFinishDocumentLoadForFrame): (WebKit::WebPageProxy::broadcastMainFrameURLChangeToOtherProcesses): (WebKit::WebPageProxy::didFinishLoadForFrame): (WebKit::WebPageProxy::didFailLoadForFrame): (WebKit::WebPageProxy::didSameDocumentNavigationForFrame): (WebKit::WebPageProxy::didSameDocumentNavigationForFrameViaJSHistoryAPI): (WebKit::WebPageProxy::didReceiveTitleForFrame): (WebKit::WebPageProxy::didFirstVisuallyNonEmptyLayoutForFrame): (WebKit::WebPageProxy::didDisplayInsecureContentForFrame): (WebKit::WebPageProxy::didRunInsecureContentForFrame): (WebKit::WebPageProxy::decidePolicyForNavigationActionSync): (WebKit::WebPageProxy::decidePolicyForNewWindowAction): (WebKit::WebPageProxy::decidePolicyForResponse): (WebKit::WebPageProxy::decidePolicyForResponseShared): (WebKit::WebPageProxy::willSubmitForm): (WebKit::WebPageProxy::didUpdateHistoryTitle): (WebKit::WebPageProxy::createNewPage): (WebKit::WebPageProxy::runJavaScriptAlert): (WebKit::WebPageProxy::runJavaScriptConfirm): (WebKit::WebPageProxy::runJavaScriptPrompt): (WebKit::WebPageProxy::runBeforeUnloadConfirmPanel): (WebKit::WebPageProxy::runOpenPanel): (WebKit::WebPageProxy::showShareSheet): (WebKit::WebPageProxy::showContactPicker): (WebKit::WebPageProxy::printFrame): (WebKit::WebPageProxy::registerEditCommandForUndo): (WebKit::WebPageProxy::showPopupMenuFromFrame): (WebKit::WebPageProxy::showPopupMenu): (WebKit::WebPageProxy::learnWord): (WebKit::WebPageProxy::ignoreWord): (WebKit::WebPageProxy::logDiagnosticMessageFromWebProcess): (WebKit::WebPageProxy::logDiagnosticMessageWithResultFromWebProcess): (WebKit::WebPageProxy::logDiagnosticMessageWithValueFromWebProcess): (WebKit::WebPageProxy::logDiagnosticMessageWithEnhancedPrivacyFromWebProcess): (WebKit::WebPageProxy::logDiagnosticMessageWithValueDictionaryFromWebProcess): (WebKit::WebPageProxy::logDiagnosticMessageWithDomainFromWebProcess): (WebKit::WebPageProxy::focusedFrameChanged): (WebKit::WebPageProxy::requestGeolocationPermissionForFrame): (WebKit::WebPageProxy::requestUserMediaPermissionForFrame): (WebKit::WebPageProxy::enumerateMediaDevicesForFrame): (WebKit::WebPageProxy::requestMediaKeySystemPermissionForFrame): (WebKit::WebPageProxy::shouldAllowDeviceOrientationAndMotionAccess): (WebKit::WebPageProxy::startURLSchemeTaskShared): (WebKit::WebPageProxy::stopURLSchemeTask): (WebKit::WebPageProxy::loadSynchronousURLSchemeTask): (WebKit::WebPageProxy::writePromisedAttachmentToPasteboard): (WebKit::WebPageProxy::requestAttachmentIcon): (WebKit::WebPageProxy::registerAttachmentIdentifierFromData): (WebKit::WebPageProxy::registerAttachmentIdentifierFromFilePath): (WebKit::WebPageProxy::registerAttachmentIdentifier): (WebKit::WebPageProxy::registerAttachmentsFromSerializedData): (WebKit::WebPageProxy::cloneAttachmentData): (WebKit::WebPageProxy::didInsertAttachmentWithIdentifier): (WebKit::WebPageProxy::didRemoveAttachmentWithIdentifier): (WebKit::WebPageProxy::didCreateSleepDisabler): (WebKit::WebPageProxy::didAccessWindowProxyPropertyViaOpenerForFrame): (WebKit::WebPageProxy::broadcastFocusedFrameToOtherProcesses): (WebKit::WebPageProxy::frameNameChanged): * Source/WebKit/UIProcess/WebPageProxyTesting.cpp: (WebKit::WebPageProxyTesting::messageSenderConnection const): * Source/WebKit/UIProcess/WebProcessProxy.cpp: (WebKit::WebProcessProxy::connectionWillOpen): (WebKit::WebProcessProxy::processWillShutDown): * Source/WebKit/UIProcess/WebsiteData/Cocoa/WebsiteDataStoreCocoa.mm: (WebKit::WebsiteDataStore::networkProcessHasEntitlementForTesting): * Source/WebKit/UIProcess/ios/WKContentView.mm: (-[WKContentView _waitForDrawToPDFCallbackForPrintFormatterIfNeeded:]): (-[WKContentView _waitForDrawToImageCallbackForPrintFormatterIfNeeded:]): * Source/WebKit/UIProcess/ios/WKContentViewInteraction.mm: (-[WKContentView ensurePositionInformationIsUpToDate:]): (-[WKContentView _handleDOMPasteRequestWithResult:]): (-[WKContentView _internalRequestAutocorrectionContextWithCompletionHandler:]): * Source/WebKit/UIProcess/ios/WebPageProxyIOS.mm: (WebKit::WebPageProxy::registerWebProcessAccessibilityToken): * Source/WebKit/UIProcess/mac/SecItemShimProxy.cpp: * Source/WebKit/UIProcess/mac/TiledCoreAnimationDrawingAreaProxy.mm: (WebKit::TiledCoreAnimationDrawingAreaProxy::waitForDidUpdateActivityState): (WebKit::TiledCoreAnimationDrawingAreaProxy::createFence): * Source/WebKit/UIProcess/mac/WKImmediateActionController.mm: (-[WKImmediateActionController immediateActionRecognizerWillBeginAnimation:]): * Source/WebKit/UIProcess/mac/WebPageProxyMac.mm: (WebKit::WebPageProxy::readSelectionFromPasteboard): (WebKit::WebPageProxy::setPromisedDataForImage): (WebKit::WebPageProxy::registerWebProcessAccessibilityToken): (WebKit::WebPageProxy::executeSavedCommandBySelector): (WebKit::WebPageProxy::acceptsFirstMouse): Canonical link: https://commits.webkit.org/282353@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