Add support for using AMD modules from extensions modules.

chrome/test/base/module_system_test.h

 // Copyright (c) 2012 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.
 
 #ifndef CHROME_TEST_BASE_MODULE_SYSTEM_TEST_H_
 #define CHROME_TEST_BASE_MODULE_SYSTEM_TEST_H_
 
 #include "chrome/renderer/extensions/chrome_v8_context.h"
 #include "extensions/renderer/module_system.h"
 #include "extensions/renderer/scoped_persistent.h"
+#include "gin/public/context_holder.h"
+#include "gin/public/isolate_holder.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "v8/include/v8.h"
 
+class ModuleSystemTestEnvironment {
+ public:
+  class AssertNatives;
+  class StringSourceMap;
+
+  explicit ModuleSystemTestEnvironment(gin::IsolateHolder* isolate_holder);
+  ~ModuleSystemTestEnvironment();
+
+  // Register a named JS module in the module system.
+  void RegisterModule(const std::string& name, const std::string& code);
+
+  // Register a named JS module with source retrieved from a ResourceBundle.
+  void RegisterModule(const std::string& name, int resource_id);
+
+  // Register a named JS module in the module system and tell the module system
+  // to use it to handle any requireNative() calls for native modules with that
+  // name.
+  void OverrideNativeHandler(const std::string& name, const std::string& code);
+
+  // Registers |file_name| from chrome/test/data/extensions as a module name
+  // |module_name|.

[not at google - send to devlin, 2014/07/07 23:18:50]

heh, we should use this more. those embedded strings are such a pita.

+  void RegisterTestFile(const std::string& module_name,
+                        const std::string& file_name);
+
+  // Create an empty object in the global scope with name |name|.
+  v8::Handle<v8::Object> CreateGlobal(const std::string& name);
+
+  void DeleteContextHolder();

[not at google - send to devlin, 2014/07/07 23:18:50]

I don't know what ContextHolder is. would it be accurate to describe this method
at a higher level as like "ShutdownGin" (or TearDownGin)? in which case it would
be nice to rename DeleteModuleSystem something similar.

[Sam McNally, 2014/07/08 00:07:04]

Done.

+
+  void DeleteModuleSystem();
+
+  extensions::ModuleSystem* module_system() {
+    return context_->module_system();
+  }
+
+  extensions::ChromeV8Context* context() { return context_.get(); }
+
+  AssertNatives* assert_natives() { return assert_natives_; }
+
+ private:
+  scoped_ptr<gin::ContextHolder> context_holder_;
+  v8::HandleScope handle_scope_;
+  scoped_ptr<extensions::ChromeV8Context> context_;
+  AssertNatives* assert_natives_;
+  scoped_ptr<StringSourceMap> source_map_;
+
+  DISALLOW_COPY_AND_ASSIGN(ModuleSystemTestEnvironment);
+};
+
 // Test fixture for testing JS that makes use of the module system.
 //
 // Typically tests will look like:
 //
 // TEST_F(MyModuleSystemTest, TestStuff) {
 //   ModuleSystem::NativesEnabledScope natives_enabled(module_system_.get());
 //   RegisterModule("test", "requireNative('assert').AssertTrue(true);");
 //   module_system_->Require("test");
 // }
 //
 // By default a test will fail if no method in the native module 'assert' is
 // called. This behaviour can be overridden by calling ExpectNoAssertionsMade().
 //
 // TODO(kalman): move this back into chrome/renderer/extensions.
 class ModuleSystemTest : public testing::Test {
  public:
   ModuleSystemTest();
   virtual ~ModuleSystemTest();
 
   virtual void TearDown() OVERRIDE;
 
  protected:
-  // Register a named JS module in the module system.
-  void RegisterModule(const std::string& name, const std::string& code);
-
-  // Register a named JS module with source retrieved from a ResourceBundle.
-  void RegisterModule(const std::string& name, int resource_id);
-
-  // Register a named JS module in the module system and tell the module system
-  // to use it to handle any requireNative() calls for native modules with that
-  // name.
-  void OverrideNativeHandler(const std::string& name, const std::string& code);
-
-  // Registers |file_name| from chrome/test/data/extensions as a module name
-  // |module_name|.
-  void RegisterTestFile(const std::string& module_name,
-                        const std::string& file_name);
-
   // Make the test fail if any asserts are called. By default a test will fail
   // if no asserts are called.
   void ExpectNoAssertionsMade();
 
-  // Create an empty object in the global scope with name |name|.
-  v8::Handle<v8::Object> CreateGlobal(const std::string& name);
+  // Runs promises that have been resolved. Resolved promises will not run
+  // until this is called.
+  void RunResolvedPromises();
 
-  v8::Isolate* isolate_;
-  v8::HandleScope handle_scope_;
-  scoped_ptr<extensions::ChromeV8Context> context_;
-  class AssertNatives;
-  AssertNatives* assert_natives_;
-  class StringSourceMap;
-  scoped_ptr<StringSourceMap> source_map_;
+  gin::IsolateHolder isolate_holder_;

[not at google - send to devlin, 2014/07/07 23:18:50]

could you expose a CreateEnvironment() and make this private?

[Sam McNally, 2014/07/08 00:07:04]

Done.

+  ModuleSystemTestEnvironment env_;

[not at google - send to devlin, 2014/07/07 23:18:50]

it would be nicer if all of these were private, and this accessed via a
protected env() call.

[Sam McNally, 2014/07/08 00:07:04]

Done.

   bool should_assertions_be_made_;
 
  private:
   DISALLOW_COPY_AND_ASSIGN(ModuleSystemTest);
 };
 
 #endif  // CHROME_TEST_BASE_MODULE_SYSTEM_TEST_H_