Service Worker: Change last update check location and HTTP cache bypass rule (2/2)

content/browser/service_worker/service_worker_register_job.cc

Index: content/browser/service_worker/service_worker_register_job.cc
diff --git a/content/browser/service_worker/service_worker_register_job.cc b/content/browser/service_worker/service_worker_register_job.cc
index 16a192858271e92bc9532362686253dc9bb308fb..cfa96728ad7554a8eee92e3f8199c834feb3f868 100644
--- a/content/browser/service_worker/service_worker_register_job.cc
+++ b/content/browser/service_worker/service_worker_register_job.cc
@@ -347,8 +347,7 @@ void ServiceWorkerRegisterJob::OnStartWorkerFinished(
     // Only bump the last check time when we've bypassed the browser cache.
     base::TimeDelta time_since_last_check =
         base::Time::Now() - registration()->last_update_check();
-    if (time_since_last_check > base::TimeDelta::FromHours(
-                                    kServiceWorkerScriptMaxCacheAgeInHours) ||
+    if (time_since_last_check > new_version()->max_age() ||

[nhiroki, 2015/08/19 06:59:01]

We could use EmbeddedWorkerInstance::network_accessed_for_script_ instead of
this calculation to check whether the cache was bypassed.

[jungkees, 2015/08/20 01:52:56]

I got it. Thanks for the point.

         new_version()->force_bypass_cache_for_scripts()) {
       registration()->set_last_update_check(base::Time::Now());
       context_->storage()->UpdateLastUpdateCheckTime(registration());