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

Unified Diff: ppapi/proxy/ppb_tcp_server_socket_private_proxy.cc

Issue 10081020: PPAPI: Make blocking completion callbacks work. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Updated TestURLLoader to test blocking callbacks. Created 8 years, 8 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: ppapi/proxy/ppb_tcp_server_socket_private_proxy.cc
diff --git a/ppapi/proxy/ppb_tcp_server_socket_private_proxy.cc b/ppapi/proxy/ppb_tcp_server_socket_private_proxy.cc
index ed5d5ad61aa5ae5f337b17b35525304ee4966c90..5e6557b61e8e88314361fa431585108ac5bc11f6 100644
--- a/ppapi/proxy/ppb_tcp_server_socket_private_proxy.cc
+++ b/ppapi/proxy/ppb_tcp_server_socket_private_proxy.cc
@@ -22,9 +22,6 @@
namespace ppapi {
namespace proxy {
-typedef thunk::EnterResource<thunk::PPB_TCPServerSocket_Private_API>
- EnterTCPServerSocket;
-
namespace {
class TCPServerSocket : public PPB_TCPServerSocket_Shared {
@@ -164,7 +161,8 @@ void PPB_TCPServerSocket_Private_Proxy::OnMsgListenACK(
PP_Resource socket_resource,
uint32 socket_id,
int32_t status) {
- EnterTCPServerSocket enter(socket_resource, true);
dmichael (off chromium) 2012/04/19 20:02:00 This is a slightly unrelated change, but it was do
+ thunk::EnterResourceNoLock<thunk::PPB_TCPServerSocket_Private_API>
+ enter(socket_resource, true);
if (enter.succeeded()) {
PPB_TCPServerSocket_Shared* server_socket =
static_cast<PPB_TCPServerSocket_Shared*>(enter.object());

Powered by Google App Engine
This is Rietveld 408576698