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

Unified Diff: third_party/WebKit/Source/core/loader/FrameFetchContext.cpp

Issue 2372563002: Adding Embedding-CSP HTTP header (Closed)
Patch Set: Moving requiredCSP to FrameLoader Created 4 years, 3 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/FrameFetchContext.cpp
diff --git a/third_party/WebKit/Source/core/loader/FrameFetchContext.cpp b/third_party/WebKit/Source/core/loader/FrameFetchContext.cpp
index 84d160747aaf9f322b4bdf23495fb1f1ba5622da..ba330754a6ac8f6dd2b8935bd71fe0ef54ba08d7 100644
--- a/third_party/WebKit/Source/core/loader/FrameFetchContext.cpp
+++ b/third_party/WebKit/Source/core/loader/FrameFetchContext.cpp
@@ -699,7 +699,9 @@ SecurityOrigin* FrameFetchContext::getSecurityOrigin() const
void FrameFetchContext::upgradeInsecureRequest(ResourceRequest& resourceRequest)
{
- frame()->loader().upgradeInsecureRequest(resourceRequest, m_document);
+ // Record the latest requiredCSP value that will be used when sending this request.
+ frame()->loader().recordLatestRequiredCSP();
+ frame()->loader().addOutgoingSecurityHeaders(resourceRequest, m_document);
Mike West 2016/09/30 13:11:49 This seems strange. I don't know why we call into
}
void FrameFetchContext::addClientHintsIfNecessary(FetchRequest& fetchRequest)

Powered by Google App Engine
This is Rietveld 408576698