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

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

Issue 2821533003: Refactor NotificationPlatformBridgeLinux (Closed)
Patch Set: final comments Created 3 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/notifications/notification_platform_bridge.h
diff --git a/chrome/browser/notifications/notification_platform_bridge.h b/chrome/browser/notifications/notification_platform_bridge.h
index 0c93da80bbf12af2a74c1946e4d44c66b2cd9cce..f90265e68b320f40673a4d87ae548e810c0c55c0 100644
--- a/chrome/browser/notifications/notification_platform_bridge.h
+++ b/chrome/browser/notifications/notification_platform_bridge.h
@@ -22,6 +22,9 @@ class Notification;
// TODO(miguelg): Add support for click and close events.
class NotificationPlatformBridge {
public:
+ using NotificationBridgeReadyCallback =
+ base::OnceCallback<void(bool /* success */)>;
+
static NotificationPlatformBridge* Create();
virtual ~NotificationPlatformBridge() {}
@@ -45,6 +48,11 @@ class NotificationPlatformBridge {
bool incognito,
const GetDisplayedNotificationsCallback& callback) const = 0;
+ // Calls |callback| once |this| is initialized. The argument is
+ // true if |this| is ready to be used and false if initialization
+ // failed. |callback| may be called directly or from a posted task.
+ virtual void SetReadyCallback(NotificationBridgeReadyCallback callback) = 0;
+
protected:
NotificationPlatformBridge() {}

Powered by Google App Engine
This is Rietveld 408576698