Implement the infrastructure of creating WorkerFetchContext in worker global scope.

third_party/WebKit/Source/web/DedicatedWorkerMessagingProxyProviderImpl.cpp

 /*
  * Copyright (C) 2009 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 13 matching lines @@
  * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
  * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
  * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
  * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
 #include "web/DedicatedWorkerMessagingProxyProviderImpl.h"
 
 #include "core/dom/Document.h"
+#include "core/loader/WorkerFetchContext.h"
 #include "core/workers/DedicatedWorkerMessagingProxy.h"
 #include "core/workers/Worker.h"
 #include "core/workers/WorkerClients.h"
 #include "platform/wtf/PtrUtil.h"
+#include "public/platform/Platform.h"
 #include "public/platform/WebContentSettingsClient.h"
 #include "public/platform/WebString.h"
 #include "public/web/WebFrameClient.h"
 #include "public/web/WebWorkerContentSettingsClientProxy.h"
 #include "web/IndexedDBClientImpl.h"
 #include "web/LocalFileSystemClient.h"
 #include "web/WebLocalFrameImpl.h"
 #include "web/WebViewImpl.h"
 #include "web/WorkerContentSettingsClient.h"
 
@@ skipping 12 matching lines @@
         WebLocalFrameImpl::FromFrame(document->GetFrame());
     WorkerClients* worker_clients = WorkerClients::Create();
     ProvideIndexedDBClientToWorker(
         worker_clients, IndexedDBClientImpl::Create(*worker_clients));
     ProvideLocalFileSystemToWorker(worker_clients,
                                    LocalFileSystemClient::Create());
     ProvideContentSettingsClientToWorker(
         worker_clients,
         WTF::WrapUnique(
             web_frame->Client()->CreateWorkerContentSettingsClientProxy()));
+    std::unique_ptr<WebWorkerFetchContext> web_worker_fetch_context =
+        Platform::Current()->CreateWorkerFetchContext();
+    // |web_worker_fetch_context| is null if off-main-thread-fetch is disabled.

[nhiroki, 2017/04/18 07:56:41]

If possible, I'd prefer to avoid checking the feature availability by a return
value of the creation method. If it's determined only by the feature flag, how
about checking it like this?

if (base::FeatureList::IsEnabled(kOffMainThreadFetch)) {
  std::unique_ptr<WebWorkerFetchContext> web_worker_fetch_context =
      Platform::Current()->CreateWorkerFetchContext();
  DCHECK(web_worker_fetch_context);
  ProvideWorkerFetchContextToWorker(
      worker_clients, std::move(web_worker_fetch_context));
}

[horo, 2017/04/18 12:53:34]

Done.

+    if (web_worker_fetch_context) {
+      // TODO(horo): Set more information about the context (ex: Service Worker
+      // provider ID) to |web_worker_fetch_context|.
+      ProvideWorkerFetchContextToWorker(worker_clients,
+                                        std::move(web_worker_fetch_context));

[kinuko, 2017/04/18 08:35:37]

I really don't think we need to use supplement here but can just pass it as
WorkerThreadStartupData or something like that, though, it might end up
requiring more changes (I quickly searched but current code setting is not very
convenient for us to do that).  It doesn't need to be addressed in this CL but
you can probably chat with nhiroki@ about that.

[horo, 2017/04/18 12:53:34]

Acknowledged.

+    }
     // FIXME: call provideServiceWorkerContainerClientToWorker here when we
     // support ServiceWorker in dedicated workers (http://crbug.com/371690)
     return new DedicatedWorkerMessagingProxy(worker, worker_clients);
   }
   NOTREACHED();
   return 0;
 }
 
 }  // namespace blink