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

Unified Diff: chrome/common/extensions/extension_manifest_constants.cc

Issue 10828363: Improve error message for insecure 'content_security_policy' manifest entries. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Aaron's sentence. Created 8 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: chrome/common/extensions/extension_manifest_constants.cc
diff --git a/chrome/common/extensions/extension_manifest_constants.cc b/chrome/common/extensions/extension_manifest_constants.cc
index ad5e4463af89a82b7a6fb50802c4e549f287ef97..beeae9b4ba0f2e4ab0c6d545d6cbbd3670ee67ce 100644
--- a/chrome/common/extensions/extension_manifest_constants.cc
+++ b/chrome/common/extensions/extension_manifest_constants.cc
@@ -452,6 +452,13 @@ const char kInvalidWebURLs[] =
"Invalid value for 'app.urls'.";
const char kInvalidZipHash[] =
"Required key 'zip_hash' is missing or invalid.";
+const char kInsecureContentSecurityPolicy[] =
+ "Invalid value for 'content_security_policy': The 'script-src' and"
+ " 'object-src' directives can only whitelist secure resources. You may"
+ " include any of the following sources: \"'self'\", \"http://127.0.0.1\","
+ " \"http://localhost\", or any \"https://\" or \"chrome-extension://\""
+ " origin. For more information, see"
+ " http://developer.chrome.com/extensions/contentSecurityPolicy.html";
const char kLaunchPathAndExtentAreExclusive[] =
"The 'app.launch.local_path' and 'app.urls' keys cannot both be set.";
const char kLaunchPathAndURLAreExclusive[] =

Powered by Google App Engine
This is Rietveld 408576698