[webkit-changes] [WebKit/WebKit] fe2039: Remove (mostly) unused AppBundle code from webpushd

Brady Eidson noreply at github.com
Tue Aug 22 18:29:41 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: fe2039934e79ead26db6ab49ec33be75b5f2e1da
      https://github.com/WebKit/WebKit/commit/fe2039934e79ead26db6ab49ec33be75b5f2e1da
  Author: Brady Eidson <beidson at apple.com>
  Date:   2023-08-22 (Tue, 22 Aug 2023)

  Changed paths:
    M Source/WebKit/NetworkProcess/NetworkProcess.cpp
    M Source/WebKit/NetworkProcess/NetworkProcess.h
    M Source/WebKit/NetworkProcess/NetworkProcess.messages.in
    M Source/WebKit/NetworkProcess/Notifications/NetworkNotificationManager.cpp
    M Source/WebKit/NetworkProcess/Notifications/NetworkNotificationManager.h
    M Source/WebKit/Shared/Notifications/NotificationManagerMessageHandler.h
    M Source/WebKit/Shared/Notifications/NotificationManagerMessageHandler.messages.in
    M Source/WebKit/Shared/WebPushDaemonConstants.h
    M Source/WebKit/UIProcess/API/Cocoa/WKWebsiteDataStore.mm
    M Source/WebKit/UIProcess/API/Cocoa/WKWebsiteDataStorePrivate.h
    M Source/WebKit/UIProcess/Network/NetworkProcessProxy.cpp
    M Source/WebKit/UIProcess/Network/NetworkProcessProxy.h
    M Source/WebKit/UIProcess/Notifications/ServiceWorkerNotificationHandler.cpp
    M Source/WebKit/UIProcess/Notifications/ServiceWorkerNotificationHandler.h
    M Source/WebKit/UIProcess/Notifications/WebNotificationManagerMessageHandler.cpp
    M Source/WebKit/UIProcess/Notifications/WebNotificationManagerMessageHandler.h
    M Source/WebKit/WebKit.xcodeproj/project.pbxproj
    M Source/WebKit/WebProcess/Notifications/NotificationPermissionRequestManager.cpp
    R Source/WebKit/webpushd/AppBundleRequest.h
    R Source/WebKit/webpushd/AppBundleRequest.mm
    R Source/WebKit/webpushd/ICAppBundle.h
    R Source/WebKit/webpushd/ICAppBundle.mm
    R Source/WebKit/webpushd/InstallCoordinationSPI.h
    R Source/WebKit/webpushd/MockAppBundleForTesting.h
    R Source/WebKit/webpushd/MockAppBundleForTesting.mm
    R Source/WebKit/webpushd/MockAppBundleRegistry.h
    R Source/WebKit/webpushd/MockAppBundleRegistry.mm
    R Source/WebKit/webpushd/PushAppBundle.h
    R Source/WebKit/webpushd/PushAppBundle.mm
    M Source/WebKit/webpushd/PushClientConnection.h
    M Source/WebKit/webpushd/PushClientConnection.mm
    M Source/WebKit/webpushd/WebPushDaemon.h
    M Source/WebKit/webpushd/WebPushDaemon.mm
    M Tools/TestWebKitAPI/Tests/WebKitCocoa/WebPushDaemon.mm

  Log Message:
  -----------
  Remove (mostly) unused AppBundle code from webpushd
https://bugs.webkit.org/show_bug.cgi?id=260549
rdar://114283091

Reviewed by Alex Christensen.

This code was part of an early design direction in webpushd that went nowhere, and only ended up
being (lightly) used for some API tests.

Removing it will simplify future refactoring.

* Source/WebKit/NetworkProcess/NetworkProcess.cpp:
(WebKit::NetworkProcess::getOriginsWithPushAndNotificationPermissions): Deleted.
* Source/WebKit/NetworkProcess/NetworkProcess.h:
* Source/WebKit/NetworkProcess/NetworkProcess.messages.in:
* Source/WebKit/NetworkProcess/Notifications/NetworkNotificationManager.cpp:
(WebKit::NetworkNotificationManager::requestSystemNotificationPermission): Deleted.
(WebKit::NetworkNotificationManager::getOriginsWithPushAndNotificationPermissions): Deleted.
* Source/WebKit/NetworkProcess/Notifications/NetworkNotificationManager.h:
* Source/WebKit/Shared/Notifications/NotificationManagerMessageHandler.h:
* Source/WebKit/Shared/Notifications/NotificationManagerMessageHandler.messages.in:
* Source/WebKit/Shared/WebPushDaemonConstants.h:
(WebKit::WebPushD::messageTypeSendsReply):
* Source/WebKit/UIProcess/API/Cocoa/WKWebsiteDataStore.mm:
(-[WKWebsiteDataStore _deletePushAndNotificationRegistration:completionHandler:]): Deleted.
(-[WKWebsiteDataStore _getOriginsWithPushAndNotificationPermissions:]): Deleted.
* Source/WebKit/UIProcess/API/Cocoa/WKWebsiteDataStorePrivate.h:
* Source/WebKit/UIProcess/Network/NetworkProcessProxy.cpp:
(WebKit::NetworkProcessProxy::getOriginsWithPushAndNotificationPermissions): Deleted.
* Source/WebKit/UIProcess/Network/NetworkProcessProxy.h:
* Source/WebKit/UIProcess/Notifications/ServiceWorkerNotificationHandler.cpp:
(WebKit::ServiceWorkerNotificationHandler::requestSystemNotificationPermission): Deleted.
* Source/WebKit/UIProcess/Notifications/ServiceWorkerNotificationHandler.h:
* Source/WebKit/UIProcess/Notifications/WebNotificationManagerMessageHandler.cpp:
(WebKit::WebNotificationManagerMessageHandler::requestSystemNotificationPermission): Deleted.
* Source/WebKit/UIProcess/Notifications/WebNotificationManagerMessageHandler.h:
* Source/WebKit/WebKit.xcodeproj/project.pbxproj:
* Source/WebKit/WebProcess/Notifications/NotificationPermissionRequestManager.cpp:
(WebKit::NotificationPermissionRequestManager::startRequest):
* Source/WebKit/webpushd/AppBundleRequest.h: Removed.
* Source/WebKit/webpushd/AppBundleRequest.mm: Removed.
* Source/WebKit/webpushd/ICAppBundle.h: Removed.
* Source/WebKit/webpushd/ICAppBundle.mm: Removed.
* Source/WebKit/webpushd/InstallCoordinationSPI.h: Removed.
* Source/WebKit/webpushd/MockAppBundleForTesting.h: Removed.
* Source/WebKit/webpushd/MockAppBundleForTesting.mm: Removed.
* Source/WebKit/webpushd/MockAppBundleRegistry.h: Removed.
* Source/WebKit/webpushd/MockAppBundleRegistry.mm: Removed.
* Source/WebKit/webpushd/PushAppBundle.h: Removed.
* Source/WebKit/webpushd/PushAppBundle.mm: Removed.
* Source/WebKit/webpushd/PushClientConnection.h:
* Source/WebKit/webpushd/PushClientConnection.mm:
(WebPushD::PushClientConnection::connectionClosed):
(WebPushD::PushClientConnection::enqueueAppBundleRequest): Deleted.
(WebPushD::PushClientConnection::maybeStartNextAppBundleRequest): Deleted.
(WebPushD::PushClientConnection::didCompleteAppBundleRequest): Deleted.
* Source/WebKit/webpushd/WebPushDaemon.h:
* Source/WebKit/webpushd/WebPushDaemon.mm:
(WebPushD::WebPushDaemon::decodeAndHandleMessage):
(WebPushD::WebPushDaemon::deletePushAndNotificationRegistration):
(WebPushD::MessageInfo::getOriginsWithPushAndNotificationPermissions::encodeReply): Deleted.
(WebPushD::MessageInfo::requestSystemNotificationPermission::encodeReply): Deleted.
(WebPushD::WebPushDaemon::requestSystemNotificationPermission): Deleted.
(WebPushD::WebPushDaemon::getOriginsWithPushAndNotificationPermissions): Deleted.
* Tools/TestWebKitAPI/Tests/WebKitCocoa/WebPushDaemon.mm:

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




More information about the webkit-changes mailing list