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

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

Issue 2372563002: Adding Embedding-CSP HTTP header (Closed)
Patch Set: Separating into two functions Created 4 years, 2 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/FrameLoader.h
diff --git a/third_party/WebKit/Source/core/loader/FrameLoader.h b/third_party/WebKit/Source/core/loader/FrameLoader.h
index e070156784df43e27622d189da8f346faf4b1468..883fba78f443e15bd72db10621371b62facdd2a3 100644
--- a/third_party/WebKit/Source/core/loader/FrameLoader.h
+++ b/third_party/WebKit/Source/core/loader/FrameLoader.h
@@ -148,11 +148,15 @@ public:
WebInsecureRequestPolicy getInsecureRequestPolicy() const;
SecurityContext::InsecureNavigationsSet* insecureNavigationsToUpgrade() const;
- void upgradeInsecureRequest(ResourceRequest&, Document*) const;
+ void addOutgoingSecurityHeadersAndUpgradeRequest(ResourceRequest&, Document*) const;
Mike West 2016/10/06 08:00:51 How about `modifyRequestForCSP`?
Frame* opener();
void setOpener(LocalFrame*);
+ const AtomicString& requiredCSP() const { return m_requiredCSP; }
+ void setRequiredCSP(const AtomicString& requiredCSP) { m_requiredCSP = requiredCSP; }
+ void recordLatestRequiredCSP();
+
void detach();
void finishedParsing();
@@ -223,10 +227,13 @@ private:
void detachDocumentLoader(Member<DocumentLoader>&);
+ void upgradeInsecureRequest(ResourceRequest&, Document*) const;
+
std::unique_ptr<TracedValue> toTracedValue() const;
void takeObjectSnapshot() const;
Member<LocalFrame> m_frame;
+ AtomicString m_requiredCSP;
// FIXME: These should be std::unique_ptr<T> to reduce build times and simplify
// header dependencies unless performance testing proves otherwise.

Powered by Google App Engine
This is Rietveld 408576698