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

Unified Diff: chrome/renderer/extensions/resource_request_policy.cc

Issue 2574763003: Remove chrome-extension-resource:// scheme (Closed)
Patch Set: Silently strip chrome-extension-resource:// from CSP directives. Created 4 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
Index: chrome/renderer/extensions/resource_request_policy.cc
diff --git a/chrome/renderer/extensions/resource_request_policy.cc b/chrome/renderer/extensions/resource_request_policy.cc
index b426f0911b67bf93811e94435bd3d9816f849fed..46460ec6f9aedf861f030408f59845f67c63b672 100644
--- a/chrome/renderer/extensions/resource_request_policy.cc
+++ b/chrome/renderer/extensions/resource_request_policy.cc
@@ -114,24 +114,4 @@ bool ResourceRequestPolicy::CanRequestResource(
return true;
}
-bool ResourceRequestPolicy::CanRequestExtensionResourceScheme(
- const blink::WebURL& resource_url,
- blink::WebFrame* frame) {
- CHECK(resource_url.protocolIs(kExtensionResourceScheme));
-
- GURL frame_url = frame->document().url();
- if (!frame_url.is_empty() && !frame_url.SchemeIs(kExtensionScheme)) {
- std::string message = base::StringPrintf(
- "Denying load of %s. chrome-extension-resources:// can only be "
- "loaded from extensions.",
- resource_url.string().utf8().c_str());
- frame->addMessageToConsole(
- blink::WebConsoleMessage(blink::WebConsoleMessage::LevelError,
- blink::WebString::fromUTF8(message)));
- return false;
- }
-
- return true;
-}
-
} // namespace extensions

Powered by Google App Engine
This is Rietveld 408576698