ServiceWorker: Database functions should return status code instead of boolean

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 <vector>
@@ skipping 50 matching lines @@
     RegistrationData();
     ~RegistrationData();
   };
 
   struct ResourceRecord {
     int64 resource_id;
     GURL url;
   };
 
   // For use during initialization.
-  bool GetNextAvailableIds(int64* next_avail_registration_id,
-                           int64* next_avail_version_id,
-                           int64* next_avail_resource_id);
-  bool GetOriginsWithRegistrations(std::set<GURL>* origins);
+  ServiceWorkerStatusCode GetNextAvailableIds(
+      int64* next_avail_registration_id,
+      int64* next_avail_version_id,
+      int64* next_avail_resource_id);
+  ServiceWorkerStatusCode GetOriginsWithRegistrations(
+      std::set<GURL>* origins);
 
   // For use when first handling a request in an origin with registrations.
   bool GetRegistrationsForOrigin(const GURL& origin,
                                  std::vector<RegistrationData>* registrations);
 
   bool GetAllRegistrations(std::vector<RegistrationData>* registrations);
 
   // Saving, retrieving, and updating registration data.
   // (will bump next_avail_xxxx_ids as needed)
   // (resource ids will be added/removed from the uncommitted/purgeable
@@ skipping 52 matching lines @@
   bool is_disabled() const { return is_disabled_; }
   bool was_corruption_detected() const { return was_corruption_detected_; }
 
  private:
   // Opens the database at the |path_|. This is lazily called when the first
   // database API is called. Returns true if the database was opened. Returns
   // false if the opening failed or was not neccessary, that is, the database
   // does not exist and |create_if_needed| is false.
   bool LazyOpen(bool create_if_needed);
 
-  bool ReadNextAvailableId(const char* id_key,
-                           int64* next_avail_id);
+  ServiceWorkerStatusCode ReadNextAvailableId(
+      const char* id_key,
+      int64* next_avail_id);
   bool ReadRegistrationData(int64 registration_id,
                             const GURL& origin,
                             RegistrationData* registration);
   bool ReadResourceRecords(int64 version_id,
                            std::vector<ResourceRecord>* resources);
   bool DeleteResourceRecords(int64 version_id,
                              leveldb::WriteBatch* batch);
   bool ReadResourceIds(const char* id_key_prefix,
                        std::set<int64>* ids);
   bool WriteResourceIds(const char* id_key_prefix,
@@ skipping 50 matching lines @@
   FRIEND_TEST_ALL_PREFIXES(ServiceWorkerDatabaseTest, Registration_Basic);
   FRIEND_TEST_ALL_PREFIXES(ServiceWorkerDatabaseTest, Registration_Overwrite);
   FRIEND_TEST_ALL_PREFIXES(ServiceWorkerDatabaseTest, Registration_Multiple);
 
   DISALLOW_COPY_AND_ASSIGN(ServiceWorkerDatabase);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_SERVICE_WORKER_SERVICE_WORKER_DATABASE_H_