Implement a module system for the extension bindings JS.

chrome/renderer/extensions/app_bindings.h

 // Copyright (c) 2011 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.
 
 // The AppObjectExtension is a v8 extension that creates an object
 // at window.chrome.app.  This object allows javascript to get details
 // on the app state of the page.
 // The read-only property app.isInstalled is true if the current page is
 // within the extent of an installed, enabled app.
 
 #ifndef CHROME_RENDERER_EXTENSIONS_APP_BINDINGS_H_
 #define CHROME_RENDERER_EXTENSIONS_APP_BINDINGS_H_
 #pragma once
 
 #include "base/compiler_specific.h"
 #include "chrome/renderer/extensions/chrome_v8_extension.h"
+#include "third_party/WebKit/Source/WebKit/chromium/public/WebFrame.h"
 
 class ChromeV8Context;
 
+using WebKit::WebFrame;
+
 // Implements the chrome.app JavaScript object.
 //
 // TODO(aa): Add unit testing for this class.
-class AppBindings : public ChromeV8Extension {
+class AppBindings : public ChromeV8Extension, public ChromeV8ExtensionHandler {

[Aaron Boodman, 2012/02/16 01:54:42]

This is multiple inheritance, which is typically discouraged in Chrome. Is there
a way to factor this such that the handler is a member of the extension?

[koz (OOO until 15th September), 2012/02/27 01:06:14]

It's possible, but it would be quite messy. It's probably better to merge
ChromeV8ExtensionHandler into ChromeV8Extension, for which I've added a TODO.

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

In that case, I prefer the multiple inheritance.

  public:
-  explicit AppBindings(ExtensionDispatcher* dispatcher);
+  explicit AppBindings(ExtensionDispatcher* dispatcher,
+      ChromeV8Context* context);
+  virtual ~AppBindings();
 
- protected:
-  virtual ChromeV8ExtensionHandler* CreateHandler(
-      ChromeV8Context* context) OVERRIDE;
+  // ChromeV8Extension
+  virtual void SetNativeFunctions(v8::Handle<v8::Object> object) OVERRIDE;
+  virtual v8::Handle<v8::Value> HandleNativeFunction(
+      const std::string& name,
+      const v8::Arguments& arguments) OVERRIDE;
+
+  // IPC::Channel::Listener
+  virtual bool OnMessageReceived(const IPC::Message& message) OVERRIDE;
 
  private:
+  v8::Handle<v8::Value> GetIsInstalled(const v8::Arguments& args);
+  v8::Handle<v8::Value> Install(const v8::Arguments& args);
+  v8::Handle<v8::Value> GetDetails(const v8::Arguments& args);
+  v8::Handle<v8::Value> GetDetailsForFrame(const v8::Arguments& args);
+  v8::Handle<v8::Value> GetAppNotifyChannel(const v8::Arguments& args);
+
+  v8::Handle<v8::Value> GetDetailsForFrameImpl(WebKit::WebFrame* frame);
+
+  void OnGetAppNotifyChannelResponse(const std::string& channel_id,
+                                     const std::string& error,
+                                     int callback_id);
+
+  ExtensionDispatcher* dispatcher_;
   DISALLOW_COPY_AND_ASSIGN(AppBindings);
 };
 
 #endif  // CHROME_RENDERER_EXTENSIONS_APP_BINDINGS_H_