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 4597cb48e86c45a197fd11d2b4c93eee0a46c7de..ab0a2fa386bac7bcb8e279e009cc6a5fcb08b8f0 100644
--- a/content/renderer/pepper/pepper_plugin_delegate_impl.h
+++ b/content/renderer/pepper/pepper_plugin_delegate_impl.h
@@ -97,6 +97,11 @@ class PepperPluginDelegateImpl
       const webkit::WebPluginInfo& webplugin_info,
       bool* pepper_plugin_was_registered);
 
+  CONTENT_EXPORT scoped_refptr<webkit::ppapi::PluginModule>

[jam, 2012/04/06 21:05:23]

nit: are you sure you need CONTENT_EXPORT (i.e. are you calling this from a unit
test? doesn't look like it, so no need to export)

[Fady Samuel, 2012/04/06 22:46:32]

Nope, remnant of an earlier revision of the code when I did somethings in chrome
instead of content. Removed.

+  CreateBrowserPluginModule(

[jam, 2012/04/06 21:05:23]

add comment for this function (see the rest of the file)

[Fady Samuel, 2012/04/06 22:46:32]

Done.

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