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

Unified Diff: content/child/notifications/notification_manager.h

Issue 1021293002: Have the browser process confirm creation of a persistent notification. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@n-db-BlowAway
Patch Set: rebase Created 5 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
Index: content/child/notifications/notification_manager.h
diff --git a/content/child/notifications/notification_manager.h b/content/child/notifications/notification_manager.h
index ae9a1ecbb8f8cadd514b94302973e7759a159bd8..7624198583cad49c5af4f8ce123c4ebcc9d3c59f 100644
--- a/content/child/notifications/notification_manager.h
+++ b/content/child/notifications/notification_manager.h
@@ -73,6 +73,7 @@ class NotificationManager : public blink::WebNotificationManager,
// IPC message handlers.
void OnDidShow(int notification_id);
+ void OnDidShowPersistent(int request_id, bool success);
void OnDidClose(int notification_id);
void OnDidClick(int notification_id);
void OnDidGetNotifications(
@@ -111,6 +112,10 @@ class NotificationManager : public blink::WebNotificationManager,
IDMap<blink::WebNotificationGetCallbacks, IDMapOwnPointer>
pending_get_notification_requests_;
+ // Tracks pending requests for displaying persistent notifications.
+ IDMap<blink::WebNotificationShowCallbacks, IDMapOwnPointer>
+ pending_show_notification_requests_;
+
// Map to store the delegate associated with a notification request Id.
std::map<int, blink::WebNotificationDelegate*> active_page_notifications_;
« no previous file with comments | « content/browser/notifications/notification_message_filter.cc ('k') | content/child/notifications/notification_manager.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698