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

Unified Diff: content/browser/renderer_host/pepper/pepper_tcp_socket_message_filter.cc

Issue 105553005: Make PepperWebPlugin not use RenderViews. (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: review comments Created 7 years 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/browser/renderer_host/pepper/pepper_tcp_socket_message_filter.cc
===================================================================
--- content/browser/renderer_host/pepper/pepper_tcp_socket_message_filter.cc (revision 242033)
+++ content/browser/renderer_host/pepper/pepper_tcp_socket_message_filter.cc (working copy)
@@ -56,7 +56,7 @@
: version_(version),
external_plugin_(host->external_plugin()),
render_process_id_(0),
- render_view_id_(0),
+ render_frame_id_(0),
ppapi_host_(host->GetPpapiHost()),
factory_(factory),
instance_(instance),
@@ -69,9 +69,9 @@
pending_accept_(false) {
DCHECK(host);
++g_num_instances;
- if (!host->GetRenderViewIDsForInstance(instance,
- &render_process_id_,
- &render_view_id_)) {
+ if (!host->GetRenderFrameIDsForInstance(instance,
+ &render_process_id_,
+ &render_frame_id_)) {
NOTREACHED();
}
}
@@ -84,7 +84,7 @@
: version_(version),
external_plugin_(host->external_plugin()),
render_process_id_(0),
- render_view_id_(0),
+ render_frame_id_(0),
ppapi_host_(host->GetPpapiHost()),
factory_(NULL),
instance_(instance),
@@ -99,9 +99,9 @@
DCHECK_NE(version, ppapi::TCP_SOCKET_VERSION_1_0);
++g_num_instances;
- if (!host->GetRenderViewIDsForInstance(instance,
- &render_process_id_,
- &render_view_id_)) {
+ if (!host->GetRenderFrameIDsForInstance(instance,
+ &render_process_id_,
+ &render_frame_id_)) {
NOTREACHED();
}
}
@@ -181,7 +181,7 @@
if (!pepper_socket_utils::CanUseSocketAPIs(
external_plugin_, false /* private_api */, NULL, render_process_id_,
- render_view_id_)) {
+ render_frame_id_)) {
return PP_ERROR_NOACCESS;
}
@@ -211,7 +211,7 @@
port);
if (!pepper_socket_utils::CanUseSocketAPIs(
external_plugin_, true /* private_api */, &request,
- render_process_id_, render_view_id_)) {
+ render_process_id_, render_frame_id_)) {
return PP_ERROR_NOACCESS;
}
@@ -241,7 +241,7 @@
content::SocketPermissionRequest::TCP_CONNECT, net_addr);
if (!pepper_socket_utils::CanUseSocketAPIs(external_plugin_, IsPrivateAPI(),
&request, render_process_id_,
- render_view_id_)) {
+ render_frame_id_)) {
return PP_ERROR_NOACCESS;
}
@@ -381,7 +381,7 @@
content::SocketPermissionRequest::TCP_LISTEN, bind_input_addr_);
if (!pepper_socket_utils::CanUseSocketAPIs(
external_plugin_, false /* private_api */, &request,
- render_process_id_, render_view_id_)) {
+ render_process_id_, render_frame_id_)) {
return PP_ERROR_NOACCESS;
}

Powered by Google App Engine
This is Rietveld 408576698