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

Unified Diff: webkit/plugins/ppapi/plugin_module.cc

Issue 19744007: Create a public API around webkit::ppapi::PluginInstance and use it in chrome. After this, webkit/p… (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: review comments and undo checkdeps change Created 7 years, 5 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
« no previous file with comments | « webkit/plugins/ppapi/plugin_module.h ('k') | webkit/plugins/ppapi/plugin_object.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webkit/plugins/ppapi/plugin_module.cc
===================================================================
--- webkit/plugins/ppapi/plugin_module.cc (revision 213462)
+++ webkit/plugins/ppapi/plugin_module.cc (working copy)
@@ -123,7 +123,7 @@
#include "webkit/plugins/ppapi/common.h"
#include "webkit/plugins/ppapi/host_globals.h"
#include "webkit/plugins/ppapi/ppapi_interface_factory.h"
-#include "webkit/plugins/ppapi/ppapi_plugin_instance.h"
+#include "webkit/plugins/ppapi/ppapi_plugin_instance_impl.h"
#include "webkit/plugins/ppapi/ppb_gpu_blacklist_private_impl.h"
#include "webkit/plugins/ppapi/ppb_image_data_impl.h"
#include "webkit/plugins/ppapi/ppb_proxy_impl.h"
@@ -239,7 +239,7 @@
}
void SimulateInputEvent(PP_Instance instance, PP_Resource input_event) {
- PluginInstance* plugin_instance = host_globals->GetInstance(instance);
+ PluginInstanceImpl* plugin_instance = host_globals->GetInstance(instance);
if (!plugin_instance)
return;
@@ -252,7 +252,7 @@
}
PP_Var GetDocumentURL(PP_Instance instance, PP_URLComponents_Dev* components) {
- PluginInstance* plugin_instance = host_globals->GetInstance(instance);
+ PluginInstanceImpl* plugin_instance = host_globals->GetInstance(instance);
if (!plugin_instance)
return PP_MakeUndefined();
return plugin_instance->GetDocumentURL(instance, components);
@@ -493,7 +493,7 @@
}
PP_ExternalPluginResult PluginModule::InitAsProxiedExternalPlugin(
- PluginInstance* instance) {
+ PluginInstanceImpl* instance) {
DCHECK(out_of_process_proxy_.get());
// InitAsProxied (for the trusted/out-of-process case) initializes only the
// module, and one or more instances are added later. In this case, the
@@ -537,13 +537,13 @@
return !!InternalGetInterface(name);
}
-PluginInstance* PluginModule::CreateInstance(
+PluginInstanceImpl* PluginModule::CreateInstance(
PluginDelegate* delegate,
content::RenderView* render_view,
WebKit::WebPluginContainer* container,
const GURL& plugin_url) {
- PluginInstance* instance = PluginInstance::Create(delegate, render_view, this,
- container, plugin_url);
+ PluginInstanceImpl* instance = PluginInstanceImpl::Create(
+ delegate, render_view, this, container, plugin_url);
if (!instance) {
LOG(WARNING) << "Plugin doesn't support instance interface, failing.";
return NULL;
@@ -553,7 +553,7 @@
return instance;
}
-PluginInstance* PluginModule::GetSomeInstance() const {
+PluginInstanceImpl* PluginModule::GetSomeInstance() const {
// This will generally crash later if there is not actually any instance to
// return, so we force a crash now to make bugs easier to track down.
CHECK(!instances_.empty());
@@ -570,11 +570,11 @@
return entry_points_.get_interface(name);
}
-void PluginModule::InstanceCreated(PluginInstance* instance) {
+void PluginModule::InstanceCreated(PluginInstanceImpl* instance) {
instances_.insert(instance);
}
-void PluginModule::InstanceDeleted(PluginInstance* instance) {
+void PluginModule::InstanceDeleted(PluginInstanceImpl* instance) {
if (out_of_process_proxy_)
out_of_process_proxy_->RemoveInstance(instance->pp_instance());
instances_.erase(instance);
« no previous file with comments | « webkit/plugins/ppapi/plugin_module.h ('k') | webkit/plugins/ppapi/plugin_object.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698