PPAPI: Refactor renderer side of browser host creation

content/public/renderer/renderer_ppapi_host.h

 // Copyright (c) 2012 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 CONTENT_PUBLIC_RENDERER_RENDERER_PPAPI_HOST_H_
 #define CONTENT_PUBLIC_RENDERER_RENDERER_PPAPI_HOST_H_
 
 #include <vector>
 
 #include "base/callback_forward.h"
 #include "base/files/file.h"
 #include "base/memory/ref_counted.h"
 #include "base/process/process.h"
 #include "content/common/content_export.h"
 #include "ipc/ipc_platform_file.h"
 #include "ppapi/c/pp_instance.h"
+#include "ppapi/proxy/serialized_structs.h"
 #include "url/gurl.h"
 
 namespace base {
 class FilePath;
 }
 
 namespace gfx {
 class Point;
 }
 
@@ skipping 15 matching lines @@
 class PepperPluginInstance;
 class RenderFrame;
 class RenderView;
 
 // Interface that allows components in the embedder app to talk to the
 // PpapiHost in the renderer process.
 //
 // There will be one of these objects in the renderer per plugin module.
 class RendererPpapiHost {
  public:
+  typedef base::Callback<void(
+      const ppapi::proxy::CompletedBrowserResourceHosts&)>
+      ResourceHostCreationCallback;
+
   // Returns the RendererPpapiHost associated with the given PP_Instance,
   // or NULL if the instance is invalid.
   //
   // Do NOT use this when dealing with an "external plugin" that serves as a
   // bootstrap to load a second plugin. This is because the two will share a
   // PP_Instance, and the RendererPpapiHost* for the second plugin will be
   // returned after we switch the proxy on.
   CONTENT_EXPORT static RendererPpapiHost* GetForPPInstance(
       PP_Instance instance);
 
@@ skipping 78 matching lines @@
   // child plugin via the Pepper proxy.
   virtual void SetToExternalPluginHost() = 0;
 
   // There are times when the renderer needs to create a ResourceHost in the
   // browser. This function does so asynchronously. |nested_msgs| is a list of
   // resource host creation messages and |instance| is the PP_Instance which
   // the resource will belong to. |callback| will be called asynchronously with
   // the pending host IDs when the ResourceHosts have been created. This can be
   // passed back to the plugin to attach to the ResourceHosts. Pending IDs of 0
   // will be passed to the callback if a ResourceHost fails to be created.
-  virtual void CreateBrowserResourceHosts(
+  //
+  // Returns a sequence number which can be used to associate responses with
+  // requests.
+  virtual int32_t CreateBrowserResourceHosts(
       PP_Instance instance,
       const std::vector<IPC::Message>& nested_msgs,
-      const base::Callback<void(const std::vector<int>&)>& callback) const = 0;
+      const ResourceHostCreationCallback& callback) const = 0;
 
   // Gets the URL of the document containing the given PP_Instance.
   // Returns an empty URL if the instance is invalid.
   // TODO(yzshen): Some methods such as this one don't need to be pure virtual.
   // Instead, they could be directly implemented using other methods in this
   // interface. Consider changing them to static helpers.
   virtual GURL GetDocumentURL(PP_Instance instance) const = 0;
 
  protected:
   virtual ~RendererPpapiHost() {}
 };
 
 }  // namespace content
 
 #endif  // CONTENT_PUBLIC_RENDERER_RENDERER_PPAPI_HOST_H_