Browser Plugin: Renderer-side changes

content/renderer/pepper/pepper_plugin_delegate_impl.h

Index: content/renderer/pepper/pepper_plugin_delegate_impl.h
diff --git a/content/renderer/pepper/pepper_plugin_delegate_impl.h b/content/renderer/pepper/pepper_plugin_delegate_impl.h
index 1d9472330cf60944bb04a987963b5efb87fbea22..31c9831c137ec5f6ed0c142b15d886722bcf8c55 100644
--- a/content/renderer/pepper/pepper_plugin_delegate_impl.h
+++ b/content/renderer/pepper/pepper_plugin_delegate_impl.h
@@ -97,6 +97,15 @@ class PepperPluginDelegateImpl
       const webkit::WebPluginInfo& webplugin_info,
       bool* pepper_plugin_was_registered);
 
+  // Creates a browser plugin instance given the process handle, and channel
+  // handle to access the guest renderer.
+  // If the plugin fails to initialize then the scoped_refptr will point

[jam, 2012/05/16 02:22:40]

nit: just say will return NULL

[Fady Samuel, 2012/05/16 04:43:54]

Done.

+  // to NULL.
+  scoped_refptr<webkit::ppapi::PluginModule>
+  CreateBrowserPluginModule(

[jam, 2012/05/16 02:22:40]

need to tab here, although it looks like you can fit this on one line

[Fady Samuel, 2012/05/16 04:43:54]

Done.

+      const IPC::ChannelHandle& channel_handle,
+      int guest_process_id);
+
   // Called by RenderView to tell us about painting events, these two functions
   // just correspond to the WillInitiatePaint, DidInitiatePaint and
   // DidFlushPaint hooks in RenderView.