Move ModuleSystemTest and the tests that use it into extensions/.

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 "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_suite.h"
+#include "content/public/test/unittest_test_suite.h"
+#include "extensions/common/extension_paths.h"
 #include "extensions/test/test_extensions_client.h"
+#include "ui/base/resource/resource_bundle.h"
 
 namespace {
 
 class ExtensionsTestSuite : public base::TestSuite {
  public:
   ExtensionsTestSuite(int argc, char** argv);
 
  private:
   // base::TestSuite:
   virtual void Initialize() OVERRIDE;
   virtual void Shutdown() OVERRIDE;
 
   scoped_ptr<extensions::TestExtensionsClient> client_;
 
   DISALLOW_COPY_AND_ASSIGN(ExtensionsTestSuite);
 };
 
 ExtensionsTestSuite::ExtensionsTestSuite(int argc, char** argv)
     : base::TestSuite(argc, argv) {}
 
 void ExtensionsTestSuite::Initialize() {
   base::TestSuite::Initialize();
 
+  extensions::RegisterPathProvider();

[tfarina, 2014/07/23 02:29:44]

are you sure you needed this? As far as I can see, it is only to register
extensions::DIR_TEST_DATA. Which you don't use here.

[Sam McNally, 2014/07/23 08:27:36]

ModuleSystemTest uses extensions::DIR_TEST_DATA.

+
+  base::FilePath resources_pack_path;
+  PathService::Get(base::DIR_MODULE, &resources_pack_path);
+  ResourceBundle::InitSharedInstanceWithPakPath(

[tfarina, 2014/07/23 02:29:44]

ui::ResourceBundle

+      resources_pack_path.AppendASCII("extensions_unittests_resources.pak"));
+
   client_.reset(new extensions::TestExtensionsClient());
   extensions::ExtensionsClient::Set(client_.get());
 }
 
 void ExtensionsTestSuite::Shutdown() {
+  ResourceBundle::CleanupSharedInstance();

[tfarina, 2014/07/23 02:29:44]

ui::ResourceBundle

   base::TestSuite::Shutdown();
 }
 
 }  // namespace
 
 int main(int argc, char** argv) {
-  ExtensionsTestSuite test_suite(argc, argv);
+  content::UnitTestTestSuite test_suite(new ExtensionsTestSuite(argc, argv));

[tfarina, 2014/07/23 02:29:43]

why content::UnitTestTestSuite here?

[Sam McNally, 2014/07/23 08:27:36]

ModuleSystemTest expects v8 to be initialized and for there to be a current
v8::Isolate. content::UnitTestTestSuite is how it's done elsewhere.

 
   return base::LaunchUnitTests(argc,
                                argv,
-                               base::Bind(&ExtensionsTestSuite::Run,
+                               base::Bind(&content::UnitTestTestSuite::Run,
                                           base::Unretained(&test_suite)));
 }