Skip to content

Commit

Permalink
Remove overzealous and incorrect RELEASE_ASSERT causing API test issues
Browse files Browse the repository at this point in the history
https://bugs.webkit.org/show_bug.cgi?id=284383
rdar://141230069

Unreviewed.

This was added in 287485@main in error.

* Source/WebKit/webpushd/WebPushDaemon.mm:
(WebPushD::WebPushDaemon::showNotification):

Canonical link: https://commits.webkit.org/287786@main
  • Loading branch information
beidson committed Dec 13, 2024
1 parent adc7380 commit 8a50c3a
Showing 1 changed file with 0 additions and 2 deletions.
2 changes: 0 additions & 2 deletions Source/WebKit/webpushd/WebPushDaemon.mm
Original file line number Diff line number Diff line change
Expand Up @@ -976,8 +976,6 @@ static bool connectionMatchesPendingPushMessage(const PushClientConnection& conn

void WebPushDaemon::showNotification(const WebCore::PushSubscriptionSetIdentifier& identifier, const WebCore::NotificationData& notificationData, RefPtr<WebCore::NotificationResources> resources, std::optional<unsigned long long> appBadge, CompletionHandler<void()>&& completionHandler)
{
RELEASE_ASSERT(supportsBuiltinNotifications(identifier));

RetainPtr content = adoptNS([[UNMutableNotificationContent alloc] init]);

[content setDefaultActionBundleIdentifier:platformDefaultActionBundleIdentifier(identifier)];
Expand Down

0 comments on commit 8a50c3a

Please sign in to comment.