Remove persistent notifications from the database when they close.

chrome/browser/notifications/platform_notification_service_impl.h

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_BROWSER_NOTIFICATIONS_PLATFORM_NOTIFICATION_SERVICE_IMPL_H_
 #define CHROME_BROWSER_NOTIFICATIONS_PLATFORM_NOTIFICATION_SERVICE_IMPL_H_
 
 #include <stdint.h>
 #include <map>
 
@@ skipping 24 matching lines @@
   // To be called when a persistent notification has been clicked on. The
   // Service Worker associated with the registration will be started if
   // needed, on which the event will be fired. Must be called on the UI thread.
   void OnPersistentNotificationClick(
       content::BrowserContext* browser_context,
       int64_t persistent_notification_id,
       const GURL& origin,
       const base::Callback<void(content::PersistentNotificationStatus)>&
           callback) const;
 
+  // To be called when a persistent notification has been closed. The data
+  // associated with the notification has to be pruned from the database in this
+  // case, to make sure that it continues to be in sync.

[johnme, 2015/04/10 11:54:39]

Nit: Must be called on the UI thread.

[Peter Beverloo, 2015/04/20 16:47:20]

Done.

+  void OnPersistentNotificationClose(
+      content::BrowserContext* browser_context,

[johnme, 2015/04/10 11:54:39]

Why not just take a Profile* since this is the chrome layer?

[Peter Beverloo, 2015/04/20 16:47:20]

It only needs to know about the BrowserContext*, so best to keep it separate. We
try to avoid introducing further dependencies on Profile*, slowly working
towards componentization.

+      int64_t persistent_notification_id,
+      const GURL& origin) const;
+
   // Returns the Notification UI Manager through which notifications can be
   // displayed to the user. Can be overridden for testing.
   NotificationUIManager* GetNotificationUIManager() const;
 
   // content::PlatformNotificationService implementation.
   blink::WebNotificationPermission CheckPermissionOnUIThread(
       content::BrowserContext* browser_context,
       const GURL& origin,
       int render_process_id) override;
   blink::WebNotificationPermission CheckPermissionOnIOThread(
@@ skipping 61 matching lines @@
   NotificationUIManager* notification_ui_manager_for_tests_;
 
   // Mapping between a persistent notification id and the id of the associated
   // message_center::Notification object. Must only be used on the UI thread.
   std::map<int64_t, std::string> persistent_notifications_;
 
   DISALLOW_COPY_AND_ASSIGN(PlatformNotificationServiceImpl);
 };
 
 #endif  // CHROME_BROWSER_NOTIFICATIONS_PLATFORM_NOTIFICATION_SERVICE_IMPL_H_