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

Unified Diff: content/plugin/webplugin_proxy.cc

Issue 1483733002: Remove support for NPObjects. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase Created 4 years, 9 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 | « content/plugin/webplugin_proxy.h ('k') | content/renderer/npapi/plugin_channel_host.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/plugin/webplugin_proxy.cc
diff --git a/content/plugin/webplugin_proxy.cc b/content/plugin/webplugin_proxy.cc
index 89c1915a5aec20946617b049315a9e7f7271fc71..ca3ddf21ee12301e2263c646bc423d43e5c719e1 100644
--- a/content/plugin/webplugin_proxy.cc
+++ b/content/plugin/webplugin_proxy.cc
@@ -8,15 +8,12 @@
#include "base/bind.h"
#include "base/lazy_instance.h"
-#include "content/child/npapi/npobject_proxy.h"
-#include "content/child/npapi/npobject_util.h"
#include "content/child/npapi/webplugin_delegate_impl.h"
#include "content/child/plugin_messages.h"
#include "content/plugin/plugin_channel.h"
#include "content/plugin/plugin_thread.h"
#include "content/public/common/content_client.h"
#include "skia/ext/platform_canvas.h"
-#include "third_party/WebKit/public/web/WebBindings.h"
#include "ui/gfx/canvas.h"
#include "url/url_constants.h"
@@ -31,8 +28,6 @@
#include "content/public/common/sandbox_init.h"
#endif
-using blink::WebBindings;
-
namespace content {
WebPluginProxy::SharedTransportDIB::SharedTransportDIB(TransportDIB* dib)
@@ -49,8 +44,6 @@ WebPluginProxy::WebPluginProxy(
int host_render_view_routing_id)
: channel_(channel),
route_id_(route_id),
- window_npobject_(NULL),
- plugin_element_(NULL),
delegate_(NULL),
waiting_for_paint_(false),
page_url_(page_url),
@@ -65,11 +58,6 @@ WebPluginProxy::~WebPluginProxy() {
if (accelerated_surface_)
accelerated_surface_.reset();
#endif
-
- if (plugin_element_)
- WebBindings::releaseObject(plugin_element_);
- if (window_npobject_)
- WebBindings::releaseObject(window_npobject_);
}
bool WebPluginProxy::Send(IPC::Message* msg) {
@@ -130,54 +118,6 @@ void WebPluginProxy::InvalidateRect(const gfx::Rect& rect) {
}
}
-NPObject* WebPluginProxy::GetWindowScriptNPObject() {
- if (window_npobject_)
- return window_npobject_;
-
- int npobject_route_id = channel_->GenerateRouteID();
- bool success = false;
- Send(new PluginHostMsg_GetWindowScriptNPObject(
- route_id_, npobject_route_id, &success));
- if (!success)
- return NULL;
-
- // PluginChannel creates a dummy owner identifier for unknown owners, so
- // use that.
- NPP owner = channel_->GetExistingNPObjectOwner(MSG_ROUTING_NONE);
-
- window_npobject_ = NPObjectProxy::Create(channel_.get(),
- npobject_route_id,
- host_render_view_routing_id_,
- page_url_,
- owner);
-
- return window_npobject_;
-}
-
-NPObject* WebPluginProxy::GetPluginElement() {
- if (plugin_element_)
- return plugin_element_;
-
- int npobject_route_id = channel_->GenerateRouteID();
- bool success = false;
- Send(new PluginHostMsg_GetPluginElement(route_id_, npobject_route_id,
- &success));
- if (!success)
- return NULL;
-
- // PluginChannel creates a dummy owner identifier for unknown owners, so
- // use that.
- NPP owner = channel_->GetExistingNPObjectOwner(MSG_ROUTING_NONE);
-
- plugin_element_ = NPObjectProxy::Create(channel_.get(),
- npobject_route_id,
- host_render_view_routing_id_,
- page_url_,
- owner);
-
- return plugin_element_;
-}
-
bool WebPluginProxy::FindProxyForUrl(const GURL& url, std::string* proxy_list) {
bool result = false;
Send(new PluginHostMsg_ResolveProxy(route_id_, url, &result, proxy_list));
« no previous file with comments | « content/plugin/webplugin_proxy.h ('k') | content/renderer/npapi/plugin_channel_host.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698