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

Unified Diff: extensions/shell/browser/shell_content_browser_client.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: extensions/shell/browser/shell_content_browser_client.cc
diff --git a/extensions/shell/browser/shell_content_browser_client.cc b/extensions/shell/browser/shell_content_browser_client.cc
index e9ee2abd2b3c44c6f53c35eb9c58ca22d88d8bcf..c3d37aec414af25971ecb31e01a1bba14f9bd64a 100644
--- a/extensions/shell/browser/shell_content_browser_client.cc
+++ b/extensions/shell/browser/shell_content_browser_client.cc
@@ -134,10 +134,9 @@ bool ShellContentBrowserClient::IsHandledURL(const GURL& url) {
url::kFileScheme,
url::kFileSystemScheme,
kExtensionScheme,
- kExtensionResourceScheme,
};
- for (size_t i = 0; i < arraysize(kProtocolList); ++i) {
- if (url.scheme() == kProtocolList[i])
+ for (const char* scheme : kProtocolList) {
+ if (url.SchemeIs(scheme))
return true;
}
return false;

Powered by Google App Engine
This is Rietveld 408576698