Index: content/browser/renderer_host/pepper/pepper_tcp_server_socket_message_filter.cc |
diff --git a/content/browser/renderer_host/pepper/pepper_tcp_server_socket_message_filter.cc b/content/browser/renderer_host/pepper/pepper_tcp_server_socket_message_filter.cc |
index 48c8291cab9889d9b5b3bb61cf801118442882ce..af2234d5110b1390acea13de94d7c33c25e63ccf 100644 |
--- a/content/browser/renderer_host/pepper/pepper_tcp_server_socket_message_filter.cc |
+++ b/content/browser/renderer_host/pepper/pepper_tcp_server_socket_message_filter.cc |
@@ -8,6 +8,7 @@ |
#include "base/bind_helpers.h" |
#include "base/logging.h" |
#include "content/browser/renderer_host/pepper/browser_ppapi_host_impl.h" |
+#include "content/browser/renderer_host/pepper/content_browser_pepper_host_factory.h" |
#include "content/browser/renderer_host/pepper/pepper_socket_utils.h" |
#include "content/public/browser/browser_thread.h" |
#include "content/public/common/socket_permission_request.h" |
@@ -20,6 +21,8 @@ |
#include "ppapi/c/private/ppb_net_address_private.h" |
#include "ppapi/host/dispatch_host_message.h" |
#include "ppapi/host/error_conversion.h" |
+#include "ppapi/host/ppapi_host.h" |
+#include "ppapi/host/resource_host.h" |
#include "ppapi/proxy/ppapi_messages.h" |
#include "ppapi/shared_impl/api_id.h" |
#include "ppapi/shared_impl/private/net_address_private_impl.h" |
@@ -36,18 +39,21 @@ size_t g_num_instances = 0; |
namespace content { |
PepperTCPServerSocketMessageFilter::PepperTCPServerSocketMessageFilter( |
+ ContentBrowserPepperHostFactory* factory, |
BrowserPpapiHostImpl* host, |
PP_Instance instance, |
- bool private_api, |
- const scoped_refptr<PepperMessageFilter>& pepper_message_filter) |
- : state_(STATE_BEFORE_LISTENING), |
- pepper_message_filter_(pepper_message_filter), |
+ bool private_api) |
+ : ppapi_host_(host->GetPpapiHost()), |
+ factory_(factory), |
+ instance_(instance), |
+ state_(STATE_BEFORE_LISTENING), |
external_plugin_(host->external_plugin()), |
private_api_(private_api), |
render_process_id_(0), |
render_view_id_(0) { |
++g_num_instances; |
- DCHECK(host); |
+ DCHECK(factory_); |
+ DCHECK(ppapi_host_); |
if (!host->GetRenderViewIDsForInstance(instance, |
&render_process_id_, |
&render_view_id_)) { |
@@ -83,7 +89,7 @@ int32_t PepperTCPServerSocketMessageFilter::OnResourceMessageReceived( |
IPC_BEGIN_MESSAGE_MAP(PepperTCPServerSocketMessageFilter, msg) |
PPAPI_DISPATCH_HOST_RESOURCE_CALL( |
PpapiHostMsg_TCPServerSocket_Listen, OnMsgListen) |
- PPAPI_DISPATCH_HOST_RESOURCE_CALL( |
+ PPAPI_DISPATCH_HOST_RESOURCE_CALL_0( |
PpapiHostMsg_TCPServerSocket_Accept, OnMsgAccept) |
PPAPI_DISPATCH_HOST_RESOURCE_CALL_0( |
PpapiHostMsg_TCPServerSocket_StopListening, OnMsgStopListening) |
@@ -117,8 +123,7 @@ int32_t PepperTCPServerSocketMessageFilter::OnMsgListen( |
} |
int32_t PepperTCPServerSocketMessageFilter::OnMsgAccept( |
- const ppapi::host::HostMessageContext* context, |
- uint32 plugin_dispatcher_id) { |
+ const ppapi::host::HostMessageContext* context) { |
DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO)); |
DCHECK(context); |
@@ -131,10 +136,9 @@ int32_t PepperTCPServerSocketMessageFilter::OnMsgAccept( |
int net_result = socket_->Accept( |
&socket_buffer_, |
base::Bind(&PepperTCPServerSocketMessageFilter::OnAcceptCompleted, |
- base::Unretained(this), |
- reply_context, plugin_dispatcher_id)); |
+ base::Unretained(this), reply_context)); |
if (net_result != net::ERR_IO_PENDING) |
- OnAcceptCompleted(reply_context, plugin_dispatcher_id, net_result); |
+ OnAcceptCompleted(reply_context, net_result); |
return PP_OK_COMPLETIONPENDING; |
} |
@@ -211,7 +215,6 @@ void PepperTCPServerSocketMessageFilter::OnListenCompleted( |
void PepperTCPServerSocketMessageFilter::OnAcceptCompleted( |
const ppapi::host::ReplyMessageContext& context, |
- uint32 plugin_dispatcher_id, |
int net_result) { |
if (state_ != STATE_ACCEPT_IN_PROGRESS) { |
SendAcceptError(context, PP_ERROR_FAILED); |
@@ -260,16 +263,18 @@ void PepperTCPServerSocketMessageFilter::OnAcceptCompleted( |
SendAcceptError(context, PP_ERROR_FAILED); |
return; |
} |
- if (!pepper_message_filter_.get() || plugin_dispatcher_id == 0) { |
- SendAcceptError(context, PP_ERROR_FAILED); |
+ |
+ scoped_ptr<ppapi::host::ResourceHost> host = |
+ factory_->CreateAcceptedTCPSocket( |
+ instance_, true /* private_api */, socket.release()); |
+ if (!host) { |
+ SendAcceptError(context, PP_ERROR_NOSPACE); |
return; |
} |
- uint32 accepted_socket_id = pepper_message_filter_->AddAcceptedTCPSocket( |
- ppapi::API_ID_PPB_TCPSOCKET_PRIVATE, |
- plugin_dispatcher_id, |
- socket.release()); |
- if (accepted_socket_id != 0) { |
- SendAcceptReply(context, PP_OK, accepted_socket_id, local_addr, |
+ int pending_resource_id = ppapi_host_->AddPendingResourceHost(host.Pass()); |
+ if (pending_resource_id) { |
+ SendAcceptReply(context, PP_OK, pending_resource_id, |
+ local_addr, |
remote_addr); |
} else { |
SendAcceptError(context, PP_ERROR_NOSPACE); |
@@ -296,13 +301,13 @@ void PepperTCPServerSocketMessageFilter::SendListenError( |
void PepperTCPServerSocketMessageFilter::SendAcceptReply( |
const ppapi::host::ReplyMessageContext& context, |
int32_t pp_result, |
- uint32 accepted_socket_id, |
+ int pending_resource_id, |
const PP_NetAddress_Private& local_addr, |
const PP_NetAddress_Private& remote_addr) { |
ppapi::host::ReplyMessageContext reply_context(context); |
reply_context.params.set_result(pp_result); |
SendReply(reply_context, PpapiPluginMsg_TCPServerSocket_AcceptReply( |
- accepted_socket_id, local_addr, remote_addr)); |
+ pending_resource_id, local_addr, remote_addr)); |
} |
void PepperTCPServerSocketMessageFilter::SendAcceptError( |