Remove the WebNotificationDelegate class
Its only responsibility is to forward calls to a NotificationHandler of
a specific type, which should instead be done as an implementation
detail of the notification system.
Bug:
Change-Id: Ia7e541c4b789b68d61209647294e420f9cde0681
Reviewed-on: https://chromium-review.googlesource.com/806220
Commit-Queue: Peter Beverloo <[email protected]>
Reviewed-by: Evan Stade <[email protected]>
Cr-Commit-Position: refs/heads/master@{#521748}diff --git a/chrome/browser/notifications/notification_common.h b/chrome/browser/notifications/notification_common.h
index 9bb8620..d83d2b2f0 100644
--- a/chrome/browser/notifications/notification_common.h
+++ b/chrome/browser/notifications/notification_common.h
@@ -9,10 +9,6 @@
#include "chrome/browser/notifications/notification_handler.h"
#include "url/gurl.h"
-namespace content {
-class BrowserContext;
-} // namespace content
-
class GURL;
class Profile;
@@ -24,7 +20,8 @@
enum Operation {
CLICK = 0,
CLOSE = 1,
- SETTINGS = 2,
+ DISABLE_PERMISSION = 2,
+ SETTINGS = 3,
OPERATION_MAX = SETTINGS
};
@@ -37,10 +34,7 @@
};
// Open the Notification settings screen when clicking the right button.
- // TODO(miguelg) have it take a Profile instead once NotificationObjectProxy
- // is updated.
- static void OpenNotificationSettings(
- content::BrowserContext* browser_context);
+ static void OpenNotificationSettings(Profile* profile, const GURL& origin);
};
// Metadata for PERSISTENT notifications.
