ServiceWorker: Schedule DeleteAndStartOver when failing to store resource ids

content/browser/service_worker/service_worker_database.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 CONTENT_BROWSER_SERVICE_WORKER_SERVICE_WORKER_DATABASE_H_
 #define CONTENT_BROWSER_SERVICE_WORKER_SERVICE_WORKER_DATABASE_H_
 
 #include <map>
 #include <set>
 #include <string>
@@ skipping 215 matching lines @@
   // Writes |ids| into the database as purgeable resources. Returns OK on
   // success. Otherwise writes nothing and returns an error.
   Status WritePurgeableResourceIds(const std::set<int64>& ids);
 
   // Deletes purgeable resource ids specified by |ids| from the database.
   // Returns OK on success. Otherwise deletes nothing and returns an error.
   Status ClearPurgeableResourceIds(const std::set<int64>& ids);
 
   // Moves |ids| from the uncommitted list to the purgeable list.
   // Returns OK on success. Otherwise deletes nothing and returns an error.
-  Status PurgeUncommittedResourceIds(const std::set<int64>& ids);
+  Status MoveResourceIdsFromUncommittedToPurgeable(const std::set<int64>& ids);
 
   // Deletes all data for |origins|, namely, unique origin, registrations and
   // resource records. Resources are moved to the purgeable list. Returns OK if
   // they are successfully deleted or not found in the database. Otherwise,
   // returns an error.
   Status DeleteAllDataForOrigins(const std::set<GURL>& origins,
                                  std::vector<int64>* newly_purgeable_resources);
 
   // Completely deletes the contents of the database.
   // Be careful using this function.
@@ skipping 44 matching lines @@
       std::vector<int64>* newly_purgeable_resources,
       leveldb::WriteBatch* batch);
 
   // Reads resource ids for |id_key_prefix| from the database. Returns OK if
   // it's successfully read or not found in the database. Otherwise, returns an
   // error.
   Status ReadResourceIds(
       const char* id_key_prefix,
       std::set<int64>* ids);
 
-  // Write resource ids for |id_key_prefix| into the database. Returns OK on
-  // success. Otherwise, returns writes nothing and returns an error.
+  // Write resource ids for |id_key_prefix| into the database and bumps the next
+  // available resource id if needed. Returns OK on success. Otherwise, returns
+  // writes nothing and returns an error.
   Status WriteResourceIds(
       const char* id_key_prefix,
       const std::set<int64>& ids);
   Status WriteResourceIdsInBatch(
       const char* id_key_prefix,
       const std::set<int64>& ids,
       leveldb::WriteBatch* batch);
 
   // Deletes resource ids for |id_key_prefix| from the database. Returns OK if
   // it's successfully deleted or not found in the database. Otherwise, returns
@@ skipping 87 matching lines @@
                            MigrateOnDiskCacheAccess);
   FRIEND_TEST_ALL_PREFIXES(ServiceWorkerDiskCacheMigratorTest,
                            NotMigrateOnDatabaseAccess);
 
   DISALLOW_COPY_AND_ASSIGN(ServiceWorkerDatabase);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_SERVICE_WORKER_SERVICE_WORKER_DATABASE_H_