Commit a31bffe1 authored by peter's avatar peter Committed by Commit bot

Remove the old signature of NotificationManager::closePersistent().

The origin now is included, and Blink has been updated to pass it in.

This is part of a three-sided patch:
  [1] https://codereview.chromium.org/982313002/
  [2] https://codereview.chromium.org/985063002/
  [3] This patch.

BUG=447628

Review URL: https://codereview.chromium.org/985693004

Cr-Commit-Position: refs/heads/master@{#320928}
parent 670a86de
...@@ -141,11 +141,6 @@ void NotificationManager::closePersistent( ...@@ -141,11 +141,6 @@ void NotificationManager::closePersistent(
base::UTF16ToUTF8(persistent_notification_id))); base::UTF16ToUTF8(persistent_notification_id)));
} }
void NotificationManager::closePersistent(
const blink::WebString& persistent_notification_id) {
closePersistent(blink::WebSerializedOrigin(), persistent_notification_id);
}
void NotificationManager::notifyDelegateDestroyed( void NotificationManager::notifyDelegateDestroyed(
blink::WebNotificationDelegate* delegate) { blink::WebNotificationDelegate* delegate) {
if (pending_notifications_.CancelPageNotificationFetches(delegate)) if (pending_notifications_.CancelPageNotificationFetches(delegate))
......
...@@ -49,9 +49,6 @@ class NotificationManager : public blink::WebNotificationManager, ...@@ -49,9 +49,6 @@ class NotificationManager : public blink::WebNotificationManager,
virtual void closePersistent( virtual void closePersistent(
const blink::WebSerializedOrigin& origin, const blink::WebSerializedOrigin& origin,
const blink::WebString& persistent_notification_id); const blink::WebString& persistent_notification_id);
// TODO(peter): Remove this method when Blink has been updated.
virtual void closePersistent(
const blink::WebString& persistent_notification_id);
virtual void notifyDelegateDestroyed( virtual void notifyDelegateDestroyed(
blink::WebNotificationDelegate* delegate); blink::WebNotificationDelegate* delegate);
virtual blink::WebNotificationPermission checkPermission( virtual blink::WebNotificationPermission checkPermission(
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment