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

Unified Diff: third_party/WebKit/Source/core/loader/WorkerFetchContext.h

Issue 2891453005: Pipe the FirstPartyForCookies to WorkerFetchContext for off-main-thread-fetch (Closed)
Patch Set: 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: third_party/WebKit/Source/core/loader/WorkerFetchContext.h
diff --git a/third_party/WebKit/Source/core/loader/WorkerFetchContext.h b/third_party/WebKit/Source/core/loader/WorkerFetchContext.h
index eac13af5fa3f1e194f2a6ebdf0ae9249ec388860..98bf8f5b004e7c5cf3c1a8c74bc459e72b0b369e 100644
--- a/third_party/WebKit/Source/core/loader/WorkerFetchContext.h
+++ b/third_party/WebKit/Source/core/loader/WorkerFetchContext.h
@@ -33,6 +33,7 @@ class WorkerFetchContext final : public BaseFetchContext {
virtual ~WorkerFetchContext();
ResourceFetcher* GetResourceFetcher();
+ KURL FirstPartyForCookies() const;
// BaseFetchContext implementation:
ContentSettingsClient* GetContentSettingsClient() const override;
@@ -62,6 +63,14 @@ class WorkerFetchContext final : public BaseFetchContext {
void AddAdditionalRequestHeaders(ResourceRequest&,
FetchResourceType) override;
+ void PopulateResourceRequest(const KURL&,
+ Resource::Type,
+ const ClientHintsPreferences&,
+ const FetchParameters::ResourceWidth&,
+ const ResourceLoaderOptions&,
+ SecurityViolationReportingPolicy,
+ ResourceRequest&) override;
falken 2017/05/17 08:55:31 I think out params as references violate modern (n
falken 2017/05/17 09:00:05 Ah, nevermind this is overridding existing code.
+ void SetFirstPartyCookieAndRequestorOrigin(ResourceRequest&) override;
DECLARE_VIRTUAL_TRACE();

Powered by Google App Engine
This is Rietveld 408576698