Implement updateViaCache flag and no-cache by default for main service worker scripts

content/browser/service_worker/service_worker_context_request_handler.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_context_request_handler.h"
 
 #include "base/time/time.h"
 #include "content/browser/service_worker/service_worker_context_core.h"
 #include "content/browser/service_worker/service_worker_provider_host.h"
 #include "content/browser/service_worker/service_worker_read_from_cache_job.h"
@@ skipping 191 matching lines @@
   if (resource_id == kInvalidServiceWorkerResourceId) {
     *out_status = CreateJobStatus::ERROR_OUT_OF_RESOURCE_IDS;
     return nullptr;
   }
 
   // Bypass the browser cache for initial installs and update checks after 24
   // hours have passed.
   int extra_load_flags = 0;
   base::TimeDelta time_since_last_check =
       base::Time::Now() - registration->last_update_check();
-  if (time_since_last_check > kServiceWorkerScriptMaxCacheAge ||
+  bool no_update_via_cache =

[nhiroki, 2017/06/21 15:04:59]

|no_update_via_cache| sounds a bit confusing. How about |should_bypass_cache|?

[yuryu, 2017/07/20 10:15:12]

Done.

+      (is_main_script && registration->update_via_cache() !=
+                             blink::WebServiceWorkerUpdateViaCache::kAll) ||
+      (!is_main_script && registration->update_via_cache() ==

[nhiroki, 2017/06/21 15:04:59]

|!is_main_script| would be unnecessary.

[yuryu, 2017/07/20 10:15:12]

I intended to reflect the spec, where algorithms for main scripts and imported
scripts are separately written.
https://w3c.github.io/ServiceWorker/#importscripts
But I agree, it is not necessary.

+                              blink::WebServiceWorkerUpdateViaCache::kNone);
+  if (no_update_via_cache ||
+      time_since_last_check > kServiceWorkerScriptMaxCacheAge ||
       version_->force_bypass_cache_for_scripts()) {
     extra_load_flags = net::LOAD_BYPASS_CACHE;
   }
 
   ServiceWorkerVersion* stored_version = registration->waiting_version()
                                              ? registration->waiting_version()
                                              : registration->active_version();
   int64_t incumbent_resource_id = kInvalidServiceWorkerResourceId;
   if (is_main_script) {
     if (stored_version && stored_version->script_url() == request->url()) {
       incumbent_resource_id =
           stored_version->script_cache_map()->LookupResourceId(request->url());
     }
     version_->embedded_worker()->OnURLJobCreatedForMainScript();
   }
   *out_status = incumbent_resource_id == kInvalidServiceWorkerResourceId
                     ? CreateJobStatus::WRITE_JOB
                     : CreateJobStatus::WRITE_JOB_WITH_INCUMBENT;
   return new ServiceWorkerWriteToCacheJob(
       request, network_delegate, resource_type_, context_, version_.get(),
       extra_load_flags, resource_id, incumbent_resource_id);
 }
 
 }  // namespace content