Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(360)

Unified Diff: content/public/renderer/renderer_ppapi_host.h

Issue 15947004: Allow renderer to create pepper ResourceHosts in the browser (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/public/renderer/renderer_ppapi_host.h
diff --git a/content/public/renderer/renderer_ppapi_host.h b/content/public/renderer/renderer_ppapi_host.h
index 4d81c50b7e80168bed5f16e0ca86e8517abeeda0..03ae07a4f749b993290354985353189f972c8463 100644
--- a/content/public/renderer/renderer_ppapi_host.h
+++ b/content/public/renderer/renderer_ppapi_host.h
@@ -23,6 +23,7 @@ class Point;
namespace IPC {
struct ChannelHandle;
+class Message;
}
namespace ppapi {
@@ -136,6 +137,18 @@ class RendererPpapiHost {
// Returns true if the plugin is running in process.
virtual bool IsRunningInProcess() const = 0;
+ // There are times when the renderer needs to create a ResourceHost in the
+ // browser. This function does so asynchronously. |nested_msg| is the
+ // resource host creation message and |instance| is the PP_Instance which
+ // the resource will belong to. |callback| will be called with the pending
+ // host ID when the ResourceHost has been created. This can be passed back
+ // to the plugin to attach to the ResourceHost. A pending ID of 0 will be
+ // passed to the callback upon error.
+ virtual void CreateBrowserResourceHost(
yzshen1 2013/05/29 17:57:21 [I need to double check the code.] If the resource
raymes 2013/06/04 04:11:11 This doesn't happen right now. I will fix it in a
+ PP_Instance instance,
+ const IPC::Message& nested_msg,
+ base::Callback<void(int)> callback) const = 0;
yzshen1 2013/05/29 17:57:21 const &?
raymes 2013/06/04 04:11:11 Done.
+
protected:
virtual ~RendererPpapiHost() {}
};

Powered by Google App Engine
This is Rietveld 408576698