Service Worker: Change the criteria for bumping the last update check time

content/browser/service_worker/service_worker_write_to_cache_job.cc

 // 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.
 
 #include "content/browser/service_worker/service_worker_write_to_cache_job.h"
 
 #include "base/bind.h"
 #include "base/callback.h"
 #include "base/strings/stringprintf.h"
 #include "base/thread_task_runner_handle.h"
@@ skipping 322 matching lines @@
           error_message);
       return;
     }
 
     if (!CheckPathRestriction(request))
       return;
 
     version_->SetMainScriptHttpResponseInfo(net_request_->response_info());
   }
 
-  if (net_request_->response_info().network_accessed)
+  if (net_request_->response_info().network_accessed &&
+      !(net_request_->response_info().was_cached))

[falken, 2015/12/07 07:40:33]

nit: I prefer to add braces to this if now that it's multi-lines

[jungkees, 2015/12/15 13:07:46]

Done.

     version_->embedded_worker()->OnNetworkAccessedForScriptLoad();
 
   http_info_.reset(new net::HttpResponseInfo(net_request_->response_info()));
   scoped_refptr<HttpResponseInfoIOBuffer> info_buffer =
       new HttpResponseInfoIOBuffer(
           new net::HttpResponseInfo(net_request_->response_info()));
   net::Error error = cache_writer_->MaybeWriteHeaders(
       info_buffer.get(),
       base::Bind(&ServiceWorkerWriteToCacheJob::OnWriteHeadersComplete,
                  weak_factory_.GetWeakPtr()));
@@ skipping 140 matching lines @@
   }
   return context_->storage()->CreateResponseReader(incumbent_resource_id_);
 }
 
 scoped_ptr<ServiceWorkerResponseWriter>
 ServiceWorkerWriteToCacheJob::CreateCacheResponseWriter() {
   return context_->storage()->CreateResponseWriter(resource_id_);
 }
 
 }  // namespace content