Implement a module system for the extension bindings JS.

chrome/renderer/module_system.cc

 // 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.
 
 #include "chrome/renderer/module_system.h"
 
 #include "base/bind.h"
-#include "chrome/renderer/static_v8_external_string_resource.h"
 #include "grit/renderer_resources.h"
 #include "ui/base/resource/resource_bundle.h"
 
 namespace {
 
-v8::Handle<v8::String> GetResource(int resourceId) {
-  const ResourceBundle& resource_bundle = ResourceBundle::GetSharedInstance();
-  return v8::String::NewExternal(new StaticV8ExternalAsciiStringResource(
-      resource_bundle.GetRawDataResource(resourceId)));
-}
-
 } // namespace
 
-ModuleSystem::ModuleSystem(const std::map<std::string, std::string>* source_map)
+ModuleSystem::ModuleSystem(
+    const std::map<std::string, base::StringPiece>* source_map)
     : source_map_(source_map) {
   RouteFunction("Run",
       base::Bind(&ModuleSystem::Run, base::Unretained(this)));
   RouteFunction("GetSource",
       base::Bind(&ModuleSystem::GetSource, base::Unretained(this)));
   RouteFunction("GetNative",
       base::Bind(&ModuleSystem::GetNative, base::Unretained(this)));
 }
 
 ModuleSystem::~ModuleSystem() {
@@ skipping 33 matching lines @@
   require_ = v8::Persistent<v8::Function>::New(
       v8::Handle<v8::Function>::Cast(require));
 }
 
 v8::Handle<v8::Value> ModuleSystem::RunString(v8::Handle<v8::String> code) {
   v8::HandleScope handle_scope;
   return handle_scope.Close(v8::Script::New(code)->Run());
 }
 
 v8::Handle<v8::Value> ModuleSystem::Run(const v8::Arguments& args) {
-  CHECK_EQ(1, args.Length());
+  CHECK_EQ(2, args.Length());
   v8::HandleScope handle_scope;
-  return handle_scope.Close(v8::Script::New(args[0]->ToString())->Run());
+  return handle_scope.Close(
+      v8::Script::New(args[0]->ToString(), args[1])->Run());
 }
 
 v8::Handle<v8::Value> ModuleSystem::GetSource(const v8::Arguments& args) {
   CHECK_EQ(1, args.Length());
   std::string module_name = *v8::String::AsciiValue(args[0]->ToString());
-  std::map<std::string, std::string>::const_iterator p =
+  std::map<std::string, base::StringPiece>::const_iterator p =
       source_map_->find(module_name);
   if (p == source_map_->end())
     return v8::Undefined();
-  return v8::String::New(p->second.c_str());
+  return ConvertString(p->second);
 }
 
 v8::Handle<v8::Value> ModuleSystem::GetNative(const v8::Arguments& args) {
   CHECK_EQ(1, args.Length());
   std::string native_name = *v8::String::AsciiValue(args[0]->ToString());
   NativeHandlerMap::iterator i = native_handler_map_.find(native_name);
   if (i == native_handler_map_.end())
     return v8::Undefined();
   return i->second->NewInstance();
 }
+
+v8::Handle<v8::String> ModuleSystem::ConvertString(
+    const base::StringPiece& string) {
+  linked_ptr<StaticV8ExternalAsciiStringResource> external_string(
+      new StaticV8ExternalAsciiStringResource(string));
+  external_strings_.push_back(external_string);
+  return v8::String::NewExternal(external_string.get());
+}
+
+v8::Handle<v8::String> ModuleSystem::GetResource(int resourceId) {
+  const ResourceBundle& resource_bundle = ResourceBundle::GetSharedInstance();
+  return ConvertString(resource_bundle.GetRawDataResource(resourceId));
+}
+