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

Unified Diff: content/browser/renderer_host/pepper/pepper_host_resolver_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_host_resolver_message_filter.cc
===================================================================
--- content/browser/renderer_host/pepper/pepper_host_resolver_message_filter.cc (revision 242033)
+++ content/browser/renderer_host/pepper/pepper_host_resolver_message_filter.cc (working copy)
@@ -86,13 +86,13 @@
: external_plugin_(host->external_plugin()),
private_api_(private_api),
render_process_id_(0),
- render_view_id_(0) {
+ render_frame_id_(0) {
DCHECK(host);
- if (!host->GetRenderViewIDsForInstance(
+ if (!host->GetRenderFrameIDsForInstance(
instance,
&render_process_id_,
- &render_view_id_)) {
+ &render_frame_id_)) {
NOTREACHED();
}
}
@@ -127,13 +127,11 @@
// Check plugin permissions.
SocketPermissionRequest request(
SocketPermissionRequest::RESOLVE_HOST, host_port.host, host_port.port);
- RenderViewHost* render_view_host =
- RenderViewHost::FromID(render_process_id_, render_view_id_);
- if (!render_view_host ||
- !pepper_socket_utils::CanUseSocketAPIs(external_plugin_,
+ if (!pepper_socket_utils::CanUseSocketAPIs(external_plugin_,
private_api_,
&request,
- render_view_host)) {
+ render_process_id_,
+ render_frame_id_)) {
return PP_ERROR_NOACCESS;
}

Powered by Google App Engine
This is Rietveld 408576698