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

content/common/service_worker/service_worker_types.h

Index: content/common/service_worker/service_worker_types.h
diff --git a/content/common/service_worker/service_worker_types.h b/content/common/service_worker/service_worker_types.h
index a2ace940c6361a298c66b92a0eb8387033ae633b..e5c1d5ea121e29fb80a05ea6fa2fe090945b3827 100644
--- a/content/common/service_worker/service_worker_types.h
+++ b/content/common/service_worker/service_worker_types.h
@@ -223,9 +223,16 @@ struct CONTENT_EXPORT ServiceWorkerRegistrationObjectInfo {
   ServiceWorkerRegistrationObjectInfo();
   int handle_id;
   GURL scope;
+  bool use_cache;

[nhiroki, 2017/03/24 00:33:42]

|scope| and |use_cache| could be replaced with ServiceWorkerRegistrationOptions
(see also my following comment in this file)

[yuryu, 2017/03/28 05:08:56]

Done.

   int64_t registration_id;
 };
 
+struct CONTENT_EXPORT ServiceWorkerRegistrationAttributes {

[nhiroki, 2017/03/24 00:33:42]

"RegistrationAttributes" could sound a bit confusing. We use "VersionAttributes"
(see line 236) for updating ServiceWorker JS-object's attributes, but this
struct isn't used for such a purpose.

Since "script_url" is a required param of register() and others (pattern,
use_cache) are defined as optional params, how about excluding "script_url" from
this struct and then renaming this to ServiceWorkerRegistrationOptions? This
name would also be consistent with the spec:
https://w3c.github.io/ServiceWorker/#dictdef-registrationoptions

[falken, 2017/03/24 04:06:19]

+1, and then using this struct when possible when passing the information to
various function.

[yuryu, 2017/03/28 05:08:56]

Done.

+  GURL pattern;
+  GURL script_url;
+  bool use_cache;
+};
+
 struct ServiceWorkerVersionAttributes {
   ServiceWorkerObjectInfo installing;
   ServiceWorkerObjectInfo waiting;