extensions/common/csp_validator.h - Issue 747403002: Ignore insecure parts of CSP in extensions and allow extension to load

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Unified Diff: extensions/common/csp_validator.h

Issue 747403002: Ignore insecure parts of CSP in extensions and allow extension to load (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master

Patch Set: small nits Created 6 years 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

« no previous file with comments | « chrome/common/extensions/manifest_tests/extension_manifests_platformapp_unittest.cc ('k') | extensions/common/csp_validator.cc » ('j') | extensions/common/csp_validator.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

Index: extensions/common/csp_validator.h

diff --git a/extensions/common/csp_validator.h b/extensions/common/csp_validator.h

index 192f898c8664c8a2db70158a5b38225ad9ab0d73..d336dda3e27768802b66980d79e675dbc148e667 100644

--- a/extensions/common/csp_validator.h

+++ b/extensions/common/csp_validator.h

@@ -41,8 +41,15 @@ enum Options {

// case for extensions. Platform apps disallow it.

// |options| is a bitmask of Options.

-bool ContentSecurityPolicyIsSecure(

- const std::string& policy, int options);

+//

+// If |sanitized_csp| is not NULL, |policy| minus the insecure values is stored

+// in |sanitized_csp|.

not at google - send to devlin 2014/12/03 20:33:05 If you're going to rename this to "SanitizeCSP" or

+// If |warnings| is not NULL, any validation errors are appended to |warnings|.

+// Returns whether |policy| meets the minimum security requirements.

not at google - send to devlin 2014/12/03 20:33:05 This happens iff there were no warnings, right? Yo

+bool ContentSecurityPolicyIsSecure(const std::string& policy,

+ int options,

+ std::string* sanitized_csp,

+ std::vector<InstallWarning>* warnings);

// Checks whether the given |policy| enforces a unique origin sandbox as

// defined by http://www.whatwg.org/specs/web-apps/current-work/multipage/