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

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

Issue 1509923002: Implement native web notifications for mac (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase Created 5 years 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/notifications/notification_ui_manager.h
diff --git a/chrome/browser/notifications/notification_ui_manager.h b/chrome/browser/notifications/notification_ui_manager.h
index d86a9a578492f206dd9c001ae2b216d42643d76c..2cd6ba58b926ae3df3ec2f511aa8bf92978325d4 100644
--- a/chrome/browser/notifications/notification_ui_manager.h
+++ b/chrome/browser/notifications/notification_ui_manager.h
@@ -45,6 +45,12 @@ class NotificationUIManager {
// crbug.com/530376
static NotificationUIManager* Create(PrefService* local_state);
+// Some platforms can integrate their notifications with their
+// own notificationn center.
Robert Sesek 2015/12/11 21:17:53 nit: indent +2
Miguel Garcia 2015/12/15 13:37:38 Done.
+#if defined(OS_MACOSX)
+ static NotificationUIManager* CreateNativeNotificationManager();
+#endif
+
// Adds a notification to be displayed. Virtual for unit test override.
virtual void Add(const Notification& notification, Profile* profile) = 0;
@@ -89,6 +95,10 @@ class NotificationUIManager {
// Used when the app is terminating.
virtual void CancelAll() = 0;
+ // Return whether the notification center can be used for native
+ // notifications.
+ virtual bool AcceptNativeNotifications() = 0;
+
protected:
NotificationUIManager() {}

Powered by Google App Engine
This is Rietveld 408576698