Beginnings of synchronizing notifications in the notification database.

content/public/browser/platform_notification_service.h

Index: content/public/browser/platform_notification_service.h
diff --git a/content/public/browser/platform_notification_service.h b/content/public/browser/platform_notification_service.h
index afcc31921f340fdc322bcb9ea7c6092848f66591..d53d7333ffb5da266faa1974878d920b26781456 100644
--- a/content/public/browser/platform_notification_service.h
+++ b/content/public/browser/platform_notification_service.h
@@ -6,6 +6,7 @@
 #define CONTENT_PUBLIC_BROWSER_PLATFORM_NOTIFICATION_SERVICE_H_
 
 #include <stdint.h>
+#include <set>
 #include <string>
 
 #include "base/callback_forward.h"
@@ -72,6 +73,13 @@ class CONTENT_EXPORT PlatformNotificationService {
   virtual void ClosePersistentNotification(
       BrowserContext* browser_context,
       int64_t persistent_notification_id) = 0;
+
+  // Writesthe ids of all currently displaying persistent notifications for the

[johnme, 2015/05/14 15:22:32]

Writesthe

[Peter Beverloo, 2015/05/14 16:03:35]

Oops. Done.

+  // given |browser_context| to |displayed_notifications|. Returns whether the
+  // platform is able to provide for such a set.

[johnme, 2015/05/14 15:22:32]

no for

[Peter Beverloo, 2015/05/14 16:03:35]

Done.

+  virtual bool GetDisplayedPersistentNotifications(
+      BrowserContext* browser_context,
+      std::set<std::string>* displayed_notifications) = 0;

[johnme, 2015/05/14 15:22:32]

Surely this should be std::set<int64_t>? Or is this intended to become a set of
tags rather than ids, in the style of your not yet landed
https://codereview.chromium.org/1133813002/ ?

[Peter Beverloo, 2015/05/14 16:03:35]

These should be strings, as is visible in my API playground CL here:
  https://codereview.chromium.org/1116693002/

The contents will indeed be the deterministic notification IDs.

 };
 
 }  // namespace content