ServiceWorker: Schedule DeleteAndStartOver when failing to store resource ids

content/browser/service_worker/service_worker_script_cache_map.cc

Index: content/browser/service_worker/service_worker_script_cache_map.cc
diff --git a/content/browser/service_worker/service_worker_script_cache_map.cc b/content/browser/service_worker/service_worker_script_cache_map.cc
index 16931391663b12a1d2749258769979e6d8df931c..3f8146ffe6b31fcf9b5ba06673a92131d712d2d8 100644
--- a/content/browser/service_worker/service_worker_script_cache_map.cc
+++ b/content/browser/service_worker/service_worker_script_cache_map.cc
@@ -48,7 +48,7 @@ void ServiceWorkerScriptCacheMap::NotifyStartedCaching(
     return;  // Our storage has been wiped via DeleteAndStartOver.
   resource_map_[url] =
       ServiceWorkerDatabase::ResourceRecord(resource_id, url, -1);
-  context_->storage()->StoreUncommittedResponseId(resource_id);
+  context_->storage()->StoreUncommittedResourceId(resource_id);

[falken, 2015/10/15 05:03:40]

Yeah, this naming has been a bit inconsistent since the beginning. See also in
service_worker_types.h:
static const int64 kInvalidServiceWorkerResourceId = -1;
static const int64 kInvalidServiceWorkerResponseId = -1;

 }
 
 void ServiceWorkerScriptCacheMap::NotifyFinishedCaching(
@@ -63,7 +63,7 @@ void ServiceWorkerScriptCacheMap::NotifyFinishedCaching(
   if (!context_)
     return;  // Our storage has been wiped via DeleteAndStartOver.
   if (!status.is_success()) {
-    context_->storage()->DoomUncommittedResponse(LookupResourceId(url));
+    context_->storage()->DoomUncommittedResource(LookupResourceId(url));
     resource_map_.erase(url);
     if (owner_->script_url() == url) {
       main_script_status_ = status;