Support DataSaver for SharedWorker

content/browser/shared_worker/shared_worker_instance.h

Index: content/browser/shared_worker/shared_worker_instance.h
diff --git a/content/browser/shared_worker/shared_worker_instance.h b/content/browser/shared_worker/shared_worker_instance.h
index 9c2359dff3abd8068aa2d8d3ae6d62b21426e40b..4e54c67cb62a13d894273304257128f542670f38 100644
--- a/content/browser/shared_worker/shared_worker_instance.h
+++ b/content/browser/shared_worker/shared_worker_instance.h
@@ -29,7 +29,8 @@ class CONTENT_EXPORT SharedWorkerInstance {
       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();
 
@@ -64,6 +65,7 @@ class CONTENT_EXPORT SharedWorkerInstance {
   blink::WebSharedWorkerCreationContextType creation_context_type() const {
     return creation_context_type_;
   }
+  bool data_saver_enabled() const { return data_saver_enabled_; }
 
  private:
   const GURL url_;
@@ -74,6 +76,7 @@ class CONTENT_EXPORT SharedWorkerInstance {
   ResourceContext* const resource_context_;
   const WorkerStoragePartitionId partition_id_;
   const blink::WebSharedWorkerCreationContextType creation_context_type_;
+  const bool data_saver_enabled_;
 };
 
 }  // namespace content