Set the requester context info to the requests for off-main-thread-fetch.

content/renderer/service_worker/worker_fetch_context_impl.h

Index: content/renderer/service_worker/worker_fetch_context_impl.h
diff --git a/content/renderer/service_worker/worker_fetch_context_impl.h b/content/renderer/service_worker/worker_fetch_context_impl.h
index 5515b045fbce95733868450867df1c8aea4583fd..6d452d2885f8271598402db886a5e97f1c00bdd2 100644
--- a/content/renderer/service_worker/worker_fetch_context_impl.h
+++ b/content/renderer/service_worker/worker_fetch_context_impl.h
@@ -19,6 +19,9 @@ namespace content {
 
 class ResourceDispatcher;
 
+// This class is used while fetching resource requests on workers (dedicated
+// worker and shared worker) when off-main-thread-fetch is enabled. This class
+// is created on the main thread and passed to the worker thread.

[falken, 2017/05/18 02:14:13]

Can you add "This class is not used for service workers. For service workers,
XXX class is used instead"?

Does it make sense for this class to be in content/renderer/service_worker/?
Should it be in content/renderer/ instead?

[horo, 2017/05/18 03:59:53]

Done.
Humm... This class inherits mojom::ServiceWorkerWorkerClient to receive
SetControllerServiceWorker() IPC.
So, it is OK to be here...
But yes, I think it is also OK to be in content/renderer/.
I don't have a strong opinion about it.

 class WorkerFetchContextImpl : public blink::WebWorkerFetchContext,
                                public mojom::ServiceWorkerWorkerClient {
  public:
@@ -37,10 +40,14 @@ class WorkerFetchContextImpl : public blink::WebWorkerFetchContext,
   // mojom::ServiceWorkerWorkerClient implementation:
   void SetControllerServiceWorker(int64_t controller_version_id) override;
 
-  // Sets the service worker status of the parent frame.
+  // Sets the fetch context status copied from the frame; the parent frame for a
+  // dedicated worker, the main frame of the shadow page for a shared worker.
   void set_service_worker_provider_id(int id);
   void set_is_controlled_by_service_worker(bool flag);
   void set_parent_frame_id(int id);
+  // Set whether the worker context is a secure context.
+  // https://www.w3.org/TR/secure-contexts/

[falken, 2017/05/18 02:14:13]

https://w3c.github.io/webappsec-secure-contexts/ is better, TR links tend to get
outdated

[horo, 2017/05/18 03:59:53]

Done.

+  void set_is_secure_context(bool flag);
 
  private:
   mojom::WorkerURLLoaderFactoryProviderPtrInfo provider_info_;
@@ -60,6 +67,7 @@ class WorkerFetchContextImpl : public blink::WebWorkerFetchContext,
 
   bool is_data_saver_enabled_ = false;
   int parent_frame_id_ = MSG_ROUTING_NONE;
+  bool is_secure_context_ = false;
 };
 
 }  // namespace content