Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(3085)

Unified Diff: content/renderer/service_worker/service_worker_fetch_context_impl.cc

Issue 2891453005: Pipe the FirstPartyForCookies to WorkerFetchContext for off-main-thread-fetch (Closed)
Patch Set: rebase Created 3 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/renderer/service_worker/service_worker_fetch_context_impl.cc
diff --git a/content/renderer/service_worker/service_worker_fetch_context_impl.cc b/content/renderer/service_worker/service_worker_fetch_context_impl.cc
index 9de7876345657359953366de74363e39a6c755f3..9be60e513b4218025d8b1bb3a8c44c4799d1b6a2 100644
--- a/content/renderer/service_worker/service_worker_fetch_context_impl.cc
+++ b/content/renderer/service_worker/service_worker_fetch_context_impl.cc
@@ -11,9 +11,11 @@
namespace content {
ServiceWorkerFetchContextImpl::ServiceWorkerFetchContextImpl(
+ const GURL& worker_script_url,
mojom::WorkerURLLoaderFactoryProviderPtrInfo provider_info,
int service_worker_provider_id)
- : provider_info_(std::move(provider_info)),
+ : worker_script_url_(worker_script_url),
+ provider_info_(std::move(provider_info)),
service_worker_provider_id_(service_worker_provider_id) {}
ServiceWorkerFetchContextImpl::~ServiceWorkerFetchContextImpl() {}
@@ -54,4 +56,12 @@ bool ServiceWorkerFetchContextImpl::IsDataSaverEnabled() const {
return is_data_saver_enabled_;
}
+blink::WebURL ServiceWorkerFetchContextImpl::FirstPartyForCookies() const {
+ // According to the spec, we can use the |worker_script_url_| for
+ // FirstPartyForCookies, because "site for cookies" for the service worker is
+ // the service worker's origin's host's registrable domain.
+ // https://tools.ietf.org/html/draft-west-first-party-cookies-07#section-2.1.2
+ return worker_script_url_;
+}
+
} // namespace content

Powered by Google App Engine
This is Rietveld 408576698