Remove chrome-extension-resource:// scheme

extensions/test/extensions_unittests_main.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include <memory>
 
 #include "base/base_paths.h"
 #include "base/bind.h"
 #include "base/macros.h"
 #include "base/path_service.h"
 #include "base/test/launcher/unit_test_launcher.h"
 #include "base/test/test_io_thread.h"
 #include "content/public/common/content_client.h"
 #include "content/public/test/content_test_suite_base.h"
 #include "content/public/test/unittest_test_suite.h"
 #include "extensions/common/constants.h"
 #include "extensions/common/extension_paths.h"
 #include "extensions/test/test_extensions_client.h"
 #include "mojo/edk/test/scoped_ipc_support.h"
 #include "ui/base/resource/resource_bundle.h"
 #include "ui/gl/test/gl_surface_test_support.h"
 #include "url/url_util.h"
 
 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
 // the url module.
 // TODO(jamescook): Should this be merged with ShellContentClient? Should this
 // be a persistent object available to tests?
 class ExtensionsContentClient : public content::ContentClient {
  public:
   ExtensionsContentClient() {}
   ~ExtensionsContentClient() override {}
 
   // content::ContentClient overrides:
   void AddAdditionalSchemes(
       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:
   DISALLOW_COPY_AND_ASSIGN(ExtensionsContentClient);
 };
 
 // The test suite for extensions_unittests.
 class ExtensionsTestSuite : public content::ContentTestSuiteBase {
  public:
   ExtensionsTestSuite(int argc, char** argv);
@@ skipping 53 matching lines @@
   content::UnitTestTestSuite test_suite(new ExtensionsTestSuite(argc, argv));
 
   base::TestIOThread test_io_thread(base::TestIOThread::kAutoStart);
   mojo::edk::test::ScopedIPCSupport ipc_support(test_io_thread.task_runner());
 
   return base::LaunchUnitTests(argc,
                                argv,
                                base::Bind(&content::UnitTestTestSuite::Run,
                                           base::Unretained(&test_suite)));
 }