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

Unified Diff: content/renderer/webplugin_delegate_proxy.cc

Issue 19761007: Move NPAPI implementation out of webkit/plugins/npapi and into content. (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: fix mac 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
Index: content/renderer/webplugin_delegate_proxy.cc
===================================================================
--- content/renderer/webplugin_delegate_proxy.cc (revision 212369)
+++ content/renderer/webplugin_delegate_proxy.cc (working copy)
@@ -25,6 +25,7 @@
#include "base/strings/utf_string_conversions.h"
#include "base/version.h"
#include "content/child/child_process.h"
+#include "content/child/npapi/webplugin.h"
#include "content/child/npobject_proxy.h"
#include "content/child/npobject_stub.h"
#include "content/child/npobject_util.h"
@@ -50,7 +51,6 @@
#include "ui/gfx/skia_util.h"
#include "webkit/common/cursors/webcursor.h"
#include "webkit/glue/webkit_glue.h"
-#include "webkit/plugins/npapi/webplugin.h"
#include "webkit/plugins/plugin_constants.h"
#include "webkit/plugins/sad_plugin.h"
@@ -99,7 +99,7 @@
// Proxy for WebPluginResourceClient. The object owns itself after creation,
// deleting itself after its callback has been called.
-class ResourceClientProxy : public webkit::npapi::WebPluginResourceClient {
+class ResourceClientProxy : public WebPluginResourceClient {
public:
ResourceClientProxy(PluginChannelHost* channel, int instance_id)
: channel_(channel), instance_id_(instance_id), resource_id_(0),
@@ -283,7 +283,7 @@
const GURL& url,
const std::vector<std::string>& arg_names,
const std::vector<std::string>& arg_values,
- webkit::npapi::WebPlugin* plugin,
+ WebPlugin* plugin,
bool load_manually) {
// TODO(shess): Attempt to work around http://crbug.com/97285 and
// http://crbug.com/141055 by retrying the connection. Reports seem
@@ -1101,8 +1101,7 @@
params.popups_allowed, params.notify_redirects);
}
-webkit::npapi::WebPluginResourceClient*
-WebPluginDelegateProxy::CreateResourceClient(
+WebPluginResourceClient* WebPluginDelegateProxy::CreateResourceClient(
unsigned long resource_id, const GURL& url, int notify_id) {
if (!channel_host_.get())
return NULL;
@@ -1113,8 +1112,7 @@
return proxy;
}
-webkit::npapi::WebPluginResourceClient*
-WebPluginDelegateProxy::CreateSeekableResourceClient(
+WebPluginResourceClient* WebPluginDelegateProxy::CreateSeekableResourceClient(
unsigned long resource_id, int range_request_id) {
if (!channel_host_.get())
return NULL;

Powered by Google App Engine
This is Rietveld 408576698