Implement a module system for the extension bindings JS.

chrome/renderer/native_handler.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/native_handler.h"
 
 #include "base/memory/linked_ptr.h"
 #include "base/logging.h"
 #include "v8/include/v8.h"
 
@@ skipping 10 matching lines @@
 // static
 v8::Handle<v8::Value> NativeHandler::Router(const v8::Arguments& args) {
   HandlerFunction* handler_function = static_cast<HandlerFunction*>(
       args.Data().As<v8::External>()->Value());
   return handler_function->Run(args);
 }
 
 void NativeHandler::RouteFunction(const std::string& name,
     const HandlerFunction& handler_function) {
   linked_ptr<HandlerFunction> function(new HandlerFunction(handler_function));
   handler_functions_.push_back(function);

[Matt Perry, 2012/02/28 20:34:20]

You use this pattern a few times. Do you know about V8's WeakReferences? You can
set up a callback for when a V8 object is garbage collected, so you can clean up
associated C++ data structures.

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

I didn't know about that, thanks. I've put a TODO to investigate using that
instead of a vector of linked_ptrs.

   v8::Handle<v8::FunctionTemplate> function_template =
       v8::FunctionTemplate::New(Router,
           v8::External::New(function.get()));
   object_template_->Set(name.c_str(), function_template);
 }
+
+void NativeHandler::RouteStaticFunction(const std::string& name,
+                                        const HandlerFunc handler_func) {
+  v8::Handle<v8::FunctionTemplate> function_template =
+      v8::FunctionTemplate::New(handler_func, v8::External::New(this));
+  object_template_->Set(name.c_str(), function_template);
+}