Implement a module system for the extension bindings JS.

chrome/renderer/extensions/chrome_v8_context.h

Index: chrome/renderer/extensions/chrome_v8_context.h
diff --git a/chrome/renderer/extensions/chrome_v8_context.h b/chrome/renderer/extensions/chrome_v8_context.h
index 11a4a87bec87d9c990898974345e32d13542e02c..52c8cf6fc74587d236eca07296eb7ff94ad02854 100644
--- a/chrome/renderer/extensions/chrome_v8_context.h
+++ b/chrome/renderer/extensions/chrome_v8_context.h
@@ -9,6 +9,7 @@
 #include <string>
 
 #include "base/basictypes.h"
+#include "chrome/renderer/module_system.h"
 #include "v8/include/v8.h"
 
 namespace WebKit {
@@ -93,6 +94,10 @@ class ChromeV8Context {
       v8::Handle<v8::Value>* argv,
       v8::Handle<v8::Value>* result) const;
 
+  void set_module_system(scoped_ptr<ModuleSystem> module_system) {

[Aaron Boodman, 2012/02/28 02:42:59]

Should go up higher w/ other setters.

[koz (OOO until 15th September), 2012/03/01 03:41:56]

Done.

+    module_system_ = module_system.Pass();
+  }
+
  private:
   // The v8 context the bindings are accessible to. We keep a strong reference
   // to it for simplicity. In the case of content scripts, this is necessary
@@ -113,6 +118,9 @@ class ChromeV8Context {
   // The type of context.
   ContextType context_type_;
 
+  // Owns and structures the JS that is injected to set up extension bindings.
+  scoped_ptr<ModuleSystem> module_system_;
+
   DISALLOW_COPY_AND_ASSIGN(ChromeV8Context);
 };