[webkit-changes] [WebKit/WebKit] cd352c: Use more smart pointers instead of raw pointers

Alex Christensen noreply at github.com
Wed Jun 14 13:31:11 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: cd352c23e71a20eb51389b4b3845293c78432a1c
      https://github.com/WebKit/WebKit/commit/cd352c23e71a20eb51389b4b3845293c78432a1c
  Author: Alex Christensen <achristensen at apple.com>
  Date:   2023-06-14 (Wed, 14 Jun 2023)

  Changed paths:
    M Source/WebCore/Modules/airplay/WebMediaSessionManager.cpp
    M Source/WebCore/Modules/airplay/WebMediaSessionManagerClient.h
    M Source/WebCore/Scripts/SettingsTemplates/InternalSettingsGenerated.h.erb
    M Source/WebCore/inspector/InspectorFrontendClientLocal.cpp
    M Source/WebCore/inspector/InspectorFrontendClientLocal.h
    M Source/WebCore/inspector/InspectorFrontendHost.h
    M Source/WebCore/page/Page.cpp
    M Source/WebCore/page/PageOverlay.cpp
    M Source/WebCore/page/PageOverlay.h
    M Source/WebCore/page/SettingsBase.cpp
    M Source/WebCore/page/SettingsBase.h
    M Source/WebCore/page/scrolling/ScrollingCoordinator.h
    M Source/WebCore/testing/InternalSettings.h
    M Source/WebKit/UIProcess/WebPageProxyInternals.h
    M Source/WebKit/UIProcess/mac/WebPageProxyMac.mm
    M Source/WebKit/WebProcess/InjectedBundle/API/c/mac/WKBundlePageBannerMac.mm
    M Source/WebKit/WebProcess/Inspector/WebInspector.cpp
    M Source/WebKit/WebProcess/Inspector/WebInspector.h
    M Source/WebKit/WebProcess/Inspector/WebInspectorClient.h
    M Source/WebKit/WebProcess/Notifications/NotificationPermissionRequestManager.h
    M Source/WebKit/WebProcess/WebCoreSupport/WebAlternativeTextClient.h
    M Source/WebKit/WebProcess/WebCoreSupport/WebColorChooser.h
    M Source/WebKit/WebProcess/WebCoreSupport/WebContextMenuClient.h
    M Source/WebKit/WebProcess/WebCoreSupport/WebDragClient.h
    M Source/WebKit/WebProcess/WebCoreSupport/WebEditorClient.cpp
    M Source/WebKit/WebProcess/WebCoreSupport/WebEditorClient.h
    M Source/WebKit/WebProcess/WebCoreSupport/WebNotificationClient.cpp
    M Source/WebKit/WebProcess/WebCoreSupport/WebNotificationClient.h
    M Source/WebKit/WebProcess/WebCoreSupport/WebPopupMenu.h
    M Source/WebKit/WebProcess/WebCoreSupport/mac/WebEditorClientMac.mm
    M Source/WebKit/WebProcess/WebPage/FindController.cpp
    M Source/WebKit/WebProcess/WebPage/FindController.h
    M Source/WebKit/WebProcess/WebPage/PageBanner.cpp
    M Source/WebKit/WebProcess/WebPage/PageBanner.h
    M Source/WebKit/WebProcess/WebPage/RemoteLayerTree/RemoteScrollingCoordinator.h
    M Source/WebKit/WebProcess/WebPage/WebBackForwardListProxy.h
    M Source/WebKit/WebProcess/WebPage/WebOpenPanelResultListener.h
    M Source/WebKit/WebProcess/WebPage/mac/PageBannerMac.mm
    M Source/WebKit/WebProcess/WebPage/mac/TiledCoreAnimationScrollingCoordinator.h
    M Source/WebKit/WebProcess/cocoa/PlaybackSessionManager.h
    M Source/WebKit/WebProcess/cocoa/VideoFullscreenManager.h
    M Source/WebKitLegacy/ios/WebCoreSupport/WebInspectorClientIOS.mm
    M Source/WebKitLegacy/mac/WebCoreSupport/WebInspectorClient.h
    M Source/WebKitLegacy/mac/WebCoreSupport/WebInspectorClient.mm
    M Source/WebKitLegacy/mac/WebView/WebMediaPlaybackTargetPicker.h
    M Source/WebKitLegacy/mac/WebView/WebMediaPlaybackTargetPicker.mm

  Log Message:
  -----------
  Use more smart pointers instead of raw pointers
https://bugs.webkit.org/show_bug.cgi?id=258046
rdar://110734344

Reviewed by David Kilzer.

* Source/WebCore/Modules/airplay/WebMediaSessionManager.cpp:
(WebCore::WebMediaSessionManager::showPlaybackTargetPicker):
* Source/WebCore/Modules/airplay/WebMediaSessionManagerClient.h:
* Source/WebCore/Scripts/SettingsTemplates/InternalSettingsGenerated.h.erb:
* Source/WebCore/inspector/InspectorFrontendClientLocal.cpp:
(WebCore::InspectorFrontendClientLocal::frontendPage):
(WebCore::InspectorFrontendClientLocal::windowObjectCleared):
* Source/WebCore/inspector/InspectorFrontendClientLocal.h:
(WebCore::InspectorFrontendClientLocal::frontendPage const): Deleted.
* Source/WebCore/inspector/InspectorFrontendHost.h:
* Source/WebCore/page/PageOverlay.cpp:
(WebCore::PageOverlay::page const):
* Source/WebCore/page/PageOverlay.h:
* Source/WebCore/page/SettingsBase.cpp:
(WebCore::SettingsBase::setStandardFontFamily):
(WebCore::SettingsBase::setFixedFontFamily):
(WebCore::SettingsBase::setSerifFontFamily):
(WebCore::SettingsBase::setSansSerifFontFamily):
(WebCore::SettingsBase::setCursiveFontFamily):
(WebCore::SettingsBase::setFantasyFontFamily):
(WebCore::SettingsBase::setPictographFontFamily):
* Source/WebCore/page/SettingsBase.h:
* Source/WebCore/page/scrolling/ScrollingCoordinator.h:
* Source/WebCore/testing/InternalSettings.h:
* Source/WebKit/UIProcess/WebPageProxyInternals.h:
* Source/WebKit/UIProcess/mac/WebPageProxyMac.mm:
(WebKit::WebPageProxy::Internals::platformView const):
* Source/WebKit/WebProcess/InjectedBundle/API/c/mac/WKBundlePageBannerMac.mm:
(WKBundlePageBannerCreateBannerWithCALayer):
* Source/WebKit/WebProcess/Inspector/WebInspector.cpp:
(WebKit::WebInspector::page const):
* Source/WebKit/WebProcess/Inspector/WebInspector.h:
(WebKit::WebInspector::page const): Deleted.
* Source/WebKit/WebProcess/Inspector/WebInspectorClient.h:
* Source/WebKit/WebProcess/Notifications/NotificationPermissionRequestManager.h:
* Source/WebKit/WebProcess/WebCoreSupport/WebAlternativeTextClient.h:
* Source/WebKit/WebProcess/WebCoreSupport/WebColorChooser.h:
* Source/WebKit/WebProcess/WebCoreSupport/WebContextMenuClient.h:
* Source/WebKit/WebProcess/WebCoreSupport/WebDragClient.h:
* Source/WebKit/WebProcess/WebCoreSupport/WebEditorClient.cpp:
(WebKit::WebEditorClient::textFieldDidBeginEditing):
(WebKit::WebEditorClient::textFieldDidEndEditing):
(WebKit::WebEditorClient::textDidChangeInTextField):
(WebKit::WebEditorClient::textDidChangeInTextArea):
(WebKit::WebEditorClient::doTextFieldCommandFromEvent):
(WebKit::WebEditorClient::textWillBeDeletedInTextField):
* Source/WebKit/WebProcess/WebCoreSupport/WebEditorClient.h:
* Source/WebKit/WebProcess/WebCoreSupport/WebNotificationClient.cpp:
(WebKit::WebNotificationClient::show):
(WebKit::WebNotificationClient::cancel):
(WebKit::WebNotificationClient::notificationObjectDestroyed):
* Source/WebKit/WebProcess/WebCoreSupport/WebNotificationClient.h:
* Source/WebKit/WebProcess/WebCoreSupport/WebPopupMenu.h:
(WebKit::WebPopupMenu::page):
* Source/WebKit/WebProcess/WebCoreSupport/mac/WebEditorClientMac.mm:
(WebKit::WebEditorClient::uppercaseWord):
(WebKit::WebEditorClient::lowercaseWord):
(WebKit::WebEditorClient::capitalizeWord):
* Source/WebKit/WebProcess/WebPage/FindController.cpp:
(WebKit::FindController::findString):
(WebKit::FindController::findStringMatches):
(WebKit::FindController::findRectsForStringMatches):
* Source/WebKit/WebProcess/WebPage/FindController.h:
* Source/WebKit/WebProcess/WebPage/PageBanner.cpp:
(WebKit::PageBanner::~PageBanner):
* Source/WebKit/WebProcess/WebPage/PageBanner.h:
(WebKit::PageBanner::Client::~Client):
* Source/WebKit/WebProcess/WebPage/RemoteLayerTree/RemoteScrollingCoordinator.h:
* Source/WebKit/WebProcess/WebPage/WebBackForwardListProxy.h:
* Source/WebKit/WebProcess/WebPage/WebOpenPanelResultListener.h:
* Source/WebKit/WebProcess/WebPage/mac/PageBannerMac.mm:
(WebKit::PageBanner::create):
(WebKit::PageBanner::PageBanner):
(WebKit::PageBanner::showIfHidden):
* Source/WebKit/WebProcess/WebPage/mac/TiledCoreAnimationScrollingCoordinator.h:
* Source/WebKit/WebProcess/cocoa/PlaybackSessionManager.h:
* Source/WebKit/WebProcess/cocoa/VideoFullscreenManager.h:
* Source/WebKitLegacy/mac/WebCoreSupport/WebInspectorClient.h:
* Source/WebKitLegacy/mac/WebCoreSupport/WebInspectorClient.mm:
(WebInspectorClient::openLocalFrontend):
(WebInspectorClient::didSetSearchingForNode):
* Source/WebKitLegacy/mac/WebView/WebMediaPlaybackTargetPicker.h:
* Source/WebKitLegacy/mac/WebView/WebMediaPlaybackTargetPicker.mm:
(WebMediaPlaybackTargetPicker::platformView const):

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




More information about the webkit-changes mailing list