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

Unified Diff: content/browser/frame_host/render_frame_host_impl.h

Issue 2764993002: CSP: group policies in didAddContentSecurityPolicy. (Closed)
Patch Set: Group the IPC instead of splitting them. (browser -> renderer) Created 3 years, 9 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/browser/frame_host/render_frame_host_impl.h
diff --git a/content/browser/frame_host/render_frame_host_impl.h b/content/browser/frame_host/render_frame_host_impl.h
index db810a9abfecbdec80719951b70f6b43169cbd81..4ab7f1a92d3e0d19bc0ace0568e5010709c49dbd 100644
--- a/content/browser/frame_host/render_frame_host_impl.h
+++ b/content/browser/frame_host/render_frame_host_impl.h
@@ -100,7 +100,6 @@ class ResourceRequestBody;
class StreamHandle;
class TimeoutMonitor;
class WebBluetoothServiceImpl;
-struct ContentSecurityPolicyHeader;
struct ContextMenuParams;
struct FileChooserParams;
struct FrameOwnerProperties;
@@ -725,12 +724,8 @@ class CONTENT_EXPORT RenderFrameHostImpl
void OnDidSetFeaturePolicyHeader(
const ParsedFeaturePolicyHeader& parsed_header);
- // A CSP |header| has been added.
- // RFC2616, section 4.2 specifies that headers appearing multiple times can be
- // combined with a comma. Hence zero, one or several |policies| are added to
- // the document.
- void OnDidAddContentSecurityPolicy(
- const ContentSecurityPolicyHeader& header,
+ // A new set of CSP |policies| has been added to the document.
+ void OnDidAddContentSecurityPolicies(
const std::vector<ContentSecurityPolicy>& policies);
void OnEnforceInsecureRequestPolicy(blink::WebInsecureRequestPolicy policy);

Powered by Google App Engine
This is Rietveld 408576698