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

Unified Diff: extensions/renderer/dispatcher.cc

Issue 1301653005: Setup for moving getUserMedia to secure origins only (Closed) Base URL: https://chromium.googlesource.com/chromium/src@master
Patch Set: Nits Created 5 years, 4 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: extensions/renderer/dispatcher.cc
diff --git a/extensions/renderer/dispatcher.cc b/extensions/renderer/dispatcher.cc
index abf92a9f0863cbe9736365d9cefb4fc1612a6225..ed64dfd9cb2f0fceff01600085ded25b5f71a224 100644
--- a/extensions/renderer/dispatcher.cc
+++ b/extensions/renderer/dispatcher.cc
@@ -210,6 +210,30 @@ Dispatcher::Dispatcher(DispatcherDelegate* delegate)
user_script_set_manager_observer_.Add(user_script_set_manager_.get());
request_sender_.reset(new RequestSender(this));
PopulateSourceMap();
+
+ // chrome-extensions: and chrome-extensions-resource: schemes should be
+ // treated as secure because communication with them is entirely in the
+ // browser, so there is no danger of manipulation or eavesdropping on
+ // communication with them by third parties.
+ WebString extension_scheme(base::ASCIIToUTF16(kExtensionScheme));
+ blink::WebSecurityPolicy::registerURLSchemeAsSecure(extension_scheme);
+
+ WebString extension_resource_scheme(base::ASCIIToUTF16(
+ kExtensionResourceScheme));
+ blink::WebSecurityPolicy::registerURLSchemeAsSecure(
+ extension_resource_scheme);
+
+ // chrome-extension: and chrome-extension-resource: resources should be
+ // allowed to receive CORS requests.
+ WebSecurityPolicy::registerURLSchemeAsCORSEnabled(extension_scheme);
+ WebSecurityPolicy::registerURLSchemeAsCORSEnabled(extension_resource_scheme);
+
+ // chrome-extension: resources should bypass Content Security Policy checks
+ // when included in protected resources.
+ WebSecurityPolicy::registerURLSchemeAsBypassingContentSecurityPolicy(
+ extension_scheme);
+ WebSecurityPolicy::registerURLSchemeAsBypassingContentSecurityPolicy(
+ extension_resource_scheme);
}
Dispatcher::~Dispatcher() {
« no previous file with comments | « chrome/renderer/chrome_content_renderer_client.cc ('k') | extensions/shell/renderer/shell_content_renderer_client.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698