Support DataSaver for SharedWorker

content/browser/shared_worker/shared_worker_instance.h

 // 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.
 
 #ifndef CONTENT_BROWSER_SHARED_WORKER_SHARED_WORKER_INSTANCE_H_
 #define CONTENT_BROWSER_SHARED_WORKER_SHARED_WORKER_INSTANCE_H_
 
 #include <string>
 
 #include "content/browser/shared_worker/worker_storage_partition.h"
@@ skipping 11 matching lines @@
 class CONTENT_EXPORT SharedWorkerInstance {
  public:
   SharedWorkerInstance(
       const GURL& url,
       const base::string16& name,
       const base::string16& content_security_policy,
       blink::WebContentSecurityPolicyType security_policy_type,
       blink::WebAddressSpace creation_address_space,
       ResourceContext* resource_context,
       const WorkerStoragePartitionId& partition_id,
-      blink::WebSharedWorkerCreationContextType creation_context_type);
+      blink::WebSharedWorkerCreationContextType creation_context_type,
+      bool data_saver_enabled);

[falken, 2017/05/17 00:54:53]

consider an enum like WebDataSaverFlag { Enabled, Disabled } or adding /*
data_saver_enabled */ comments to the callsites.

[kinuko, 2017/05/17 02:13:15]

+1

[horo, 2017/05/17 02:15:42]

Done.

[kinuko, 2017/05/17 06:13:22]

I know it's a bit of hassle, but could we use enum?

   SharedWorkerInstance(const SharedWorkerInstance& other);
   ~SharedWorkerInstance();
 
   // Checks if this SharedWorkerInstance matches the passed url/name params
   // based on the algorithm in the WebWorkers spec - an instance matches if the
   // origins of the URLs match, and:
   // a) the names are non-empty and equal.
   // -or-
   // b) the names are both empty, and the urls are equal.
   bool Matches(const GURL& url,
@@ skipping 14 matching lines @@
   blink::WebAddressSpace creation_address_space() const {
     return creation_address_space_;
   }
   ResourceContext* resource_context() const {
     return resource_context_;
   }
   const WorkerStoragePartitionId& partition_id() const { return partition_id_; }
   blink::WebSharedWorkerCreationContextType creation_context_type() const {
     return creation_context_type_;
   }
+  bool data_saver_enabled() const { return data_saver_enabled_; }
 
  private:
   const GURL url_;
   const base::string16 name_;
   const base::string16 content_security_policy_;
   const blink::WebContentSecurityPolicyType security_policy_type_;
   const blink::WebAddressSpace creation_address_space_;
   ResourceContext* const resource_context_;
   const WorkerStoragePartitionId partition_id_;
   const blink::WebSharedWorkerCreationContextType creation_context_type_;
+  const bool data_saver_enabled_;
 };
 
 }  // namespace content
 
 
 #endif  // CONTENT_BROWSER_SHARED_WORKER_SHARED_WORKER_INSTANCE_H_