Eagerly delete replaced notifications from the database

content/browser/notifications/notification_database.h

 // Copyright 2015 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 CONTENT_BROWSER_NOTIFICATIONS_NOTIFICATION_DATABASE_H_
 #define CONTENT_BROWSER_NOTIFICATIONS_NOTIFICATION_DATABASE_H_
 
 #include <stdint.h>
 
 #include <memory>
@@ skipping 101 matching lines @@
       const GURL& origin,
       const NotificationDatabaseData& notification_database_data,
       int64_t* notification_id);
 
   // Deletes all data associated with the notification identified by
   // |notification_id| belonging to |origin| from the database. Returns the
   // status code of the deletion operation. Note that it is not considered a
   // failure if the to-be-deleted notification does not exist.
   Status DeleteNotificationData(int64_t notification_id, const GURL& origin);
 
-  // Deletes all data associated with |origin| from the database, and appends
-  // the deleted notification ids to |deleted_notification_set|. Returns the
-  // status code of the deletion operation.
+  // Deletes all data associated with |origin| from the database, optionally
+  // filtered by the |tag|, and appends the deleted notification ids to
+  // |deleted_notification_set|. Returns the status code of the deletion
+  // operation.
   Status DeleteAllNotificationDataForOrigin(
       const GURL& origin,
+      const std::string& tag,
       std::set<int64_t>* deleted_notification_set);
 
   // Deletes all data associated with the |service_worker_registration_id|
   // belonging to |origin| from the database, and appends the deleted
   // notification ids to |deleted_notification_set|. Returns the status code
   // of the deletion operation.
   Status DeleteAllNotificationDataForServiceWorkerRegistration(
       const GURL& origin,
       int64_t service_worker_registration_id,
       std::set<int64_t>* deleted_notification_set);
@@ skipping 23 matching lines @@
   // data for |origin| will be read. If both are set, then all notification data
   // for the given |service_worker_registration_id| will be read.
   Status ReadAllNotificationDataInternal(
       const GURL& origin,
       int64_t service_worker_registration_id,
       std::vector<NotificationDatabaseData>* notification_data_vector) const;
 
   // Deletes all notification data with the given constraints. |origin| must
   // always be set - use Destroy() when the goal is to empty the database. If
   // |service_worker_registration_id| is invalid, all notification data for the
-  // |origin| will be deleted.
+  // |origin| will be deleted, optionally filtered by the |tag| when non-empty.
   // All deleted notification ids will be written to |deleted_notification_set|.
   Status DeleteAllNotificationDataInternal(
       const GURL& origin,
+      const std::string& tag,
       int64_t service_worker_registration_id,
       std::set<int64_t>* deleted_notification_set);
 
   // Returns whether the database has been opened.
   bool IsOpen() const { return db_ != nullptr; }
 
   // Returns whether the database should only exist in memory.
   bool IsInMemoryDatabase() const { return path_.empty(); }
 
   // Exposes the LevelDB database used to back this notification database.
@@ skipping 14 matching lines @@
   State state_ = STATE_UNINITIALIZED;
 
   base::SequenceChecker sequence_checker_;
 
   DISALLOW_COPY_AND_ASSIGN(NotificationDatabase);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_NOTIFICATIONS_NOTIFICATION_DATABASE_H_