Pipe the FirstPartyForCookies to WorkerFetchContext for off-main-thread-fetch

content/renderer/shared_worker/embedded_shared_worker_stub.cc

 // 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.
 
 #include "content/renderer/shared_worker/embedded_shared_worker_stub.h"
 
 #include <stdint.h>
 #include <utility>
 
 #include "base/feature_list.h"
@@ skipping 254 matching lines @@
     blink::WebServiceWorkerNetworkProvider* web_network_provider) {
   DCHECK(base::FeatureList::IsEnabled(features::kOffMainThreadFetch));
   mojom::WorkerURLLoaderFactoryProviderPtr worker_url_loader_factory_provider;
   RenderThreadImpl::current()
       ->blink_platform_impl()
       ->GetInterfaceProvider()
       ->GetInterface(mojo::MakeRequest(&worker_url_loader_factory_provider));
   std::unique_ptr<WorkerFetchContextImpl> worker_fetch_context =
       base::MakeUnique<WorkerFetchContextImpl>(
           worker_url_loader_factory_provider.PassInterface());
+  // TODO(horo): To get the correct first_party_to_cookies for the shared
+  // worker, we need to check the all documents bounded by the shared worker.
+  // https://tools.ietf.org/html/draft-west-first-party-cookies-07#section-2.1.2

[falken, 2017/05/17 08:55:31]

This sounds serious. Shall we open a bug for this?

[horo, 2017/05/17 09:44:03]

Filed crbug.com/723553

+  worker_fetch_context->set_first_party_for_cookies(url_);
   if (web_network_provider) {
     ServiceWorkerNetworkProvider* network_provider =
         ServiceWorkerNetworkProvider::FromWebServiceWorkerNetworkProvider(
             web_network_provider);
     worker_fetch_context->set_service_worker_provider_id(
         network_provider->provider_id());
     worker_fetch_context->set_is_controlled_by_service_worker(
         network_provider->IsControlledByServiceWorker());
   }
   return std::move(worker_fetch_context);
@@ skipping 32 matching lines @@
   }
 }
 
 void EmbeddedSharedWorkerStub::OnTerminateWorkerContext() {
   // After this we wouldn't get any IPC for this stub.
   running_ = false;
   impl_->TerminateWorkerContext();
 }
 
 }  // namespace content