Bind plugin placeholder directly to v8 instead of over CppBoundClass

components/plugins/renderer/plugin_placeholder.h

 // Copyright 2013 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 COMPONENTS_PLUGINS_RENDERER_PLUGIN_PLACEHOLDER_H_
 #define COMPONENTS_PLUGINS_RENDERER_PLUGIN_PLACEHOLDER_H_
 
 #include "components/plugins/renderer/webview_plugin.h"
 #include "content/public/common/webplugininfo.h"
 #include "content/public/renderer/context_menu_client.h"
 #include "content/public/renderer/render_process_observer.h"
 #include "content/public/renderer/render_view_observer.h"
 #include "third_party/WebKit/public/web/WebPluginParams.h"
-#include "webkit/renderer/cpp_bound_class.h"
+#include "v8/include/v8.h"
 
 namespace content {
 struct WebPluginInfo;
 }
 
 namespace plugins {
 // Placeholders can be used if a plug-in is missing or not available
 // (blocked or disabled).
 class PluginPlaceholder : public content::RenderViewObserver,
-                          public webkit_glue::CppBoundClass,
                           public WebViewPlugin::Delegate {
  public:
 
   WebViewPlugin* plugin() { return plugin_; }
 
   void set_blocked_for_prerendering(bool blocked_for_prerendering) {
     is_blocked_for_prerendering_ = blocked_for_prerendering;
   }
 
   void set_allow_loading(bool allow_loading) { allow_loading_ = allow_loading; }
 
  protected:
   // |render_view| and |frame| are weak pointers. If either one is going away,
   // our |plugin_| will be destroyed as well and will notify us.
   PluginPlaceholder(content::RenderView* render_view,
                     blink::WebFrame* frame,
                     const blink::WebPluginParams& params,
                     const std::string& html_data,
                     GURL placeholderDataUrl);
 
   virtual ~PluginPlaceholder();
 
+  // Derived classes can override this method to install additional V8
+  // properties on the prototype of the window.plugin object.
+  virtual void InstallAdditionalCallbacks(v8::Handle<v8::Template> prototype);

[Bernhard Bauer, 2013/11/12 16:37:20]

Super-nit: From the description of the method it seems you can install arbitrary
properties on the window.plugin object, but the method name just mentions
callbacks. Update one or the other?

+
   void OnLoadBlockedPlugins(const std::string& identifier);
   void OnSetIsPrerendering(bool is_prerendering);
 
   void SetMessage(const string16& message);
   void SetPluginInfo(const content::WebPluginInfo& plugin_info);
   const content::WebPluginInfo& GetPluginInfo() const;
   void SetIdentifier(const std::string& identifier);
   blink::WebFrame* GetFrame();
   const blink::WebPluginParams& GetPluginParams() const;
   bool LoadingAllowed() const { return allow_loading_; }
@@ skipping 10 matching lines @@
 
   // WebViewPlugin::Delegate method:
   virtual void BindWebFrame(blink::WebFrame* frame) OVERRIDE;
 
  private:
   // WebViewPlugin::Delegate methods:
   virtual void WillDestroyPlugin() OVERRIDE;
   virtual void ShowContextMenu(const blink::WebMouseEvent&) OVERRIDE;
 
   // Javascript callbacks:
-  // All ignore arguments (which are, however, required by caller) and return
-  // nothing.
 
   // Load the blocked plugin by calling LoadPlugin().
-  void LoadCallback(const webkit_glue::CppArgumentList& args,
-                    webkit_glue::CppVariant* result);
+  static void LoadCallback(const v8::FunctionCallbackInfo<v8::Value>& args);
 
   // Hide the blocked plugin by calling HidePlugin().
-  void HideCallback(const webkit_glue::CppArgumentList& args,
-                    webkit_glue::CppVariant* result);
+  static void HideCallback(const v8::FunctionCallbackInfo<v8::Value>& args);
 
-  void DidFinishLoadingCallback(const webkit_glue::CppArgumentList& args,
-                                webkit_glue::CppVariant* result);
+  static void DidFinishLoadingCallback(
+      const v8::FunctionCallbackInfo<v8::Value>& args);
 
   void UpdateMessage();
 
   blink::WebFrame* frame_;
   blink::WebPluginParams plugin_params_;
   WebViewPlugin* plugin_;
 
   content::WebPluginInfo plugin_info_;
 
   string16 message_;
 
   // True iff the plugin was blocked because the page was being prerendered.
   // Plugin will automatically be loaded when the page is displayed.
   bool is_blocked_for_prerendering_;
   bool allow_loading_;
 
   bool hidden_;
   bool finished_loading_;
   std::string identifier_;
 
   DISALLOW_COPY_AND_ASSIGN(PluginPlaceholder);
 };
 
 }  // namespace plugins
 
 #endif  // COMPONENTS_PLUGINS_RENDERER_PLUGIN_PLACEHOLDER_H_