Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(6769)

Unified Diff: chrome/browser/notifications/desktop_notification_service.h

Issue 12631017: Remove notifications when apps and OTR Profiles are removed. (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Created 7 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | chrome/browser/notifications/desktop_notification_service.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/notifications/desktop_notification_service.h
diff --git a/chrome/browser/notifications/desktop_notification_service.h b/chrome/browser/notifications/desktop_notification_service.h
index 16739b13dee41d512fb713a00c09a192dffc3a6f..4ccd8735a07c2cbefe89b88781910d48dc1f54d1 100644
--- a/chrome/browser/notifications/desktop_notification_service.h
+++ b/chrome/browser/notifications/desktop_notification_service.h
@@ -40,8 +40,7 @@ class Image;
// The DesktopNotificationService is an object, owned by the Profile,
// which provides the creation of desktop "toasts" to web pages and workers.
-class DesktopNotificationService : public content::NotificationObserver,
- public ProfileKeyedService {
+class DesktopNotificationService : public ProfileKeyedService {
public:
enum DesktopNotificationSource {
PageNotification,
@@ -83,11 +82,6 @@ class DesktopNotificationService : public content::NotificationObserver,
void GrantPermission(const GURL& origin);
void DenyPermission(const GURL& origin);
- // content::NotificationObserver implementation.
- virtual void Observe(int type,
- const content::NotificationSource& source,
- const content::NotificationDetails& details) OVERRIDE;
-
// Creates a data:xxxx URL which contains the full HTML for a notification
// using supplied icon, title, and text, run through a template which contains
// the standard formatting for notifications.
@@ -161,9 +155,6 @@ class DesktopNotificationService : public content::NotificationObserver,
void SetExtensionEnabled(const std::string& id, bool enabled);
private:
- void StartObserving();
- void StopObserving();
-
// Takes a notification object and shows it in the UI.
void ShowNotification(const Notification& notification);
@@ -187,8 +178,6 @@ class DesktopNotificationService : public content::NotificationObserver,
// UI for desktop toasts.
NotificationUIManager* ui_manager_;
- content::NotificationRegistrar notification_registrar_;
-
// Prefs listener for disabled_extension_id.
StringListPrefMember disabled_extension_id_pref_;
« no previous file with comments | « no previous file | chrome/browser/notifications/desktop_notification_service.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698