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

Unified Diff: chrome/browser/browser_process.h

Issue 1814923002: Nuke NotificationUIManager from PlatformNotificationServiceImpl (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@profile_manager_load
Patch Set: Review comments + unique_ptr rename Created 4 years, 8 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
Index: chrome/browser/browser_process.h
diff --git a/chrome/browser/browser_process.h b/chrome/browser/browser_process.h
index b38070761553c43c092eb8b06363a9518dd1d534..90310c99a0e20385259697242e11472284f29d5d 100644
--- a/chrome/browser/browser_process.h
+++ b/chrome/browser/browser_process.h
@@ -30,6 +30,7 @@ class IconManager;
class IntranetRedirectDetector;
class IOThread;
class MediaFileSystemRegistry;
+class NotificationBridge;
class NotificationUIManager;
class PrefRegistrySimple;
class PrefService;
@@ -150,7 +151,10 @@ class BrowserProcess {
extension_event_router_forwarder() = 0;
// Returns the manager for desktop notifications.
+ // TODO(miguelg) This is in the process of being deprecated in favour of
+ // NotificationBridge + NotificationDisplayService
virtual NotificationUIManager* notification_ui_manager() = 0;
+ virtual NotificationBridge* notification_bridge() = 0;
// MessageCenter is a global list of currently displayed notifications.
virtual message_center::MessageCenter* message_center() = 0;

Powered by Google App Engine
This is Rietveld 408576698