Refactoring of new IPC-only pepper implementation

content/renderer/pepper/pepper_in_process_resource_creation.h

Index: content/renderer/pepper/pepper_in_process_resource_creation.h
diff --git a/content/renderer/pepper/pepper_in_process_resource_creation.h b/content/renderer/pepper/pepper_in_process_resource_creation.h
index 6fb6f90cce972a76b1f726b665b76b8522a4f056..01fdfeb98976b277932c4004fb0e2ff04ad4f17e 100644
--- a/content/renderer/pepper/pepper_in_process_resource_creation.h
+++ b/content/renderer/pepper/pepper_in_process_resource_creation.h
@@ -7,28 +7,25 @@
 
 #include "base/basictypes.h"
 #include "base/memory/scoped_ptr.h"
-#include "content/renderer/pepper/pepper_instance_state_accessor_impl.h"
 #include "ppapi/proxy/connection.h"
 #include "webkit/plugins/ppapi/resource_creation_impl.h"
 
-class RenderViewImpl;
-
-namespace ppapi {
-class PpapiPermissions;
-}
-
 namespace content {
 
+class RendererPpapiHostImpl;
+
 // This class provides creation functions for the new resources with IPC
 // backends that live in content/renderer/pepper.
 //
+// (See pepper_in_process_router.h for more information.)
+//
 // This is a bit confusing. The "old-style" resources live in
 // webkit/plugins/ppapi and are created by the ResourceCreationImpl in that
 // directory. The "new-style" IPC-only resources are in ppapi/proxy and are
 // created by the RessourceCreationProxy in that directory.
 //
-// This class allows us to run new-style IPC-only resources in-process. We have
-// an IPC reflector to run it in process. But then we have a problem with
+// This class allows us to run new-style IPC-only resources in-process. We use the

[raymes, 2012/07/25 16:53:22]

long line

+// PepperInProcessRouter to run it in process. But then we have a problem with
 // allocating the resources since src/webkit can't depend on IPC or see our IPC
 // backend in content. This class overrides the normal in-process resource
 // creation and adds in the resources that we implement in ppapi/proxy.
@@ -39,9 +36,8 @@ namespace content {
 class PepperInProcessResourceCreation
     : public webkit::ppapi::ResourceCreationImpl {
  public:
-  PepperInProcessResourceCreation(RenderViewImpl* render_view,
-                                  webkit::ppapi::PluginInstance* instance,
-                                  const ppapi::PpapiPermissions& perms);
+  PepperInProcessResourceCreation(RendererPpapiHostImpl* host_impl,
+                                  webkit::ppapi::PluginInstance* instance);
   virtual ~PepperInProcessResourceCreation();
 
   // ResourceCreation_API implementation.
@@ -51,21 +47,8 @@ class PepperInProcessResourceCreation
       const char* accept_types) OVERRIDE;
 
  private:
-  ppapi::proxy::Connection GetConnection();
-
-  PepperInstanceStateAccessorImpl instance_state_;
-
-  // Fake channel to the browser for resource constructors.
-  class DummyBrowserChannel;
-  scoped_ptr<DummyBrowserChannel> dummy_browser_channel_;
-
-  // Renderer -> plugin channel.
-  class HostToPluginRouter;
-  scoped_ptr<HostToPluginRouter> host_to_plugin_router_;
-
-  // Plugin -> renderer channel.
-  class PluginToHostRouter;
-  scoped_ptr<PluginToHostRouter> plugin_to_host_router_;
+  // Non-owning pointer to the host for the current plugin.
+  RendererPpapiHostImpl* host_impl_;
 
   DISALLOW_COPY_AND_ASSIGN(PepperInProcessResourceCreation);
 };