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

Unified Diff: extensions/test/extensions_unittests_main.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/test/extensions_unittests_main.cc
diff --git a/extensions/test/extensions_unittests_main.cc b/extensions/test/extensions_unittests_main.cc
index c7df11011be9137eb20427c3e32d351bbbe3857f..6e2d18b1c3f08523ff241b26db159db988f76e7e 100644
--- a/extensions/test/extensions_unittests_main.cc
+++ b/extensions/test/extensions_unittests_main.cc
@@ -23,11 +23,8 @@
namespace {
-const int kNumExtensionStandardURLSchemes = 2;
-const url::SchemeWithType kExtensionStandardURLSchemes[
- kNumExtensionStandardURLSchemes] = {
- {extensions::kExtensionScheme, url::SCHEME_WITHOUT_PORT},
- {extensions::kExtensionResourceScheme, url::SCHEME_WITHOUT_PORT},
+const url::SchemeWithType kExtensionStandardURLSchemes[] = {
James Cook 2016/12/14 20:41:50 ditto
ncarter (slow) 2016/12/14 22:26:42 Done.
+ {extensions::kExtensionScheme, url::SCHEME_WITHOUT_PORT},
};
// Content client that exists only to register chrome-extension:// scheme with
@@ -44,11 +41,9 @@ class ExtensionsContentClient : public content::ContentClient {
std::vector<url::SchemeWithType>* standard_schemes,
std::vector<url::SchemeWithType>* referrer_schemes,
std::vector<std::string>* savable_schemes) override {
- for (int i = 0; i < kNumExtensionStandardURLSchemes; i++)
- standard_schemes->push_back(kExtensionStandardURLSchemes[i]);
-
+ for (const url::SchemeWithType& scheme : kExtensionStandardURLSchemes)
+ standard_schemes->push_back(scheme);
savable_schemes->push_back(extensions::kExtensionScheme);
- savable_schemes->push_back(extensions::kExtensionResourceScheme);
}
private:

Powered by Google App Engine
This is Rietveld 408576698