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

Side by Side Diff: chrome/browser/notifications/notification_display_service.h

Issue 2093953002: Introduce a new API to handle native notification clicks (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: nits Created 4 years, 5 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 unified diff | Download patch
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CHROME_BROWSER_NOTIFICATIONS_NOTIFICATION_DISPLAY_SERVICE_H_ 5 #ifndef CHROME_BROWSER_NOTIFICATIONS_NOTIFICATION_DISPLAY_SERVICE_H_
6 #define CHROME_BROWSER_NOTIFICATIONS_NOTIFICATION_DISPLAY_SERVICE_H_ 6 #define CHROME_BROWSER_NOTIFICATIONS_NOTIFICATION_DISPLAY_SERVICE_H_
7 7
8 #include <set> 8 #include <set>
9 #include <string> 9 #include <string>
10
10 #include "base/macros.h" 11 #include "base/macros.h"
12 #include "chrome/browser/notifications/notification_common.h"
11 #include "components/keyed_service/core/keyed_service.h" 13 #include "components/keyed_service/core/keyed_service.h"
12 14
13 class Notification; 15 class Notification;
14 class NotificationPlatformBridge; 16 class NotificationPlatformBridge;
15 class Profile; 17 class Profile;
16 18
17 // Profile-bound service that enables notifications to be displayed and 19 // Profile-bound service that enables notifications to be displayed and
18 // interacted with on the user's screen, orthogonal of whether this 20 // interacted with on the user's screen, orthogonal of whether this
19 // functionality is provided by the browser or by the operating system. An 21 // functionality is provided by the browser or by the operating system. An
20 // instance can be retrieved through the NotificationDisplayServiceFactory. 22 // instance can be retrieved through the NotificationDisplayServiceFactory.
21 // 23 //
22 // TODO(peter): Add a NotificationHandler mechanism for registering listeners. 24 // TODO(peter): Add a NotificationHandler mechanism for registering listeners.
23 // TODO(miguelg): Remove the SupportsNotificationCenter method. 25 // TODO(miguelg): Remove the SupportsNotificationCenter method.
24 class NotificationDisplayService : public KeyedService { 26 class NotificationDisplayService : public KeyedService {
25 public: 27 public:
26 NotificationDisplayService() {} 28 NotificationDisplayService() {}
27 ~NotificationDisplayService() override {} 29 ~NotificationDisplayService() override {}
28 30
29 // Displays the |notification| identified by |notification_id|. 31 // Displays the |notification| identified by |notification_id|.
30 virtual void Display(const std::string& notification_id, 32 virtual void Display(NotificationCommon::Type notification_type,
33 const std::string& notification_id,
31 const Notification& notification) = 0; 34 const Notification& notification) = 0;
32 35
33 // Closes the notification identified by |notification_id|. 36 // Closes the notification identified by |notification_id|.
34 virtual void Close(const std::string& notification_id) = 0; 37 virtual void Close(NotificationCommon::Type notification_type,
38 const std::string& notification_id) = 0;
35 39
36 // Returns whether the implementation can retrieve a list of currently visible 40 // Returns whether the implementation can retrieve a list of currently visible
37 // notifications and stores them in |*notification_ids| when possible. 41 // notifications and stores them in |*notification_ids| when possible.
38 virtual bool GetDisplayed(std::set<std::string>* notifications) const = 0; 42 virtual bool GetDisplayed(std::set<std::string>* notifications) const = 0;
39 43
40 // Temporary method while we finish the refactor. Returns whether there is 44 // Temporary method while we finish the refactor. Returns whether there is
41 // a native notification center backing up notifications. 45 // a native notification center backing up notifications.
42 virtual bool SupportsNotificationCenter() const = 0; 46 virtual bool SupportsNotificationCenter() const = 0;
43 47
44 private: 48 private:
45 DISALLOW_COPY_AND_ASSIGN(NotificationDisplayService); 49 DISALLOW_COPY_AND_ASSIGN(NotificationDisplayService);
46 }; 50 };
47 51
48 #endif // CHROME_BROWSER_NOTIFICATIONS_NOTIFICATION_DISPLAY_SERVICE_H_ 52 #endif // CHROME_BROWSER_NOTIFICATIONS_NOTIFICATION_DISPLAY_SERVICE_H_
OLDNEW
« no previous file with comments | « chrome/browser/notifications/notification_common.h ('k') | chrome/browser/notifications/notification_handler.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698