Never lock the Pepper proxy lock on the IO thread

content/browser/renderer_host/pepper/pepper_udp_socket_message_filter.cc

Index: content/browser/renderer_host/pepper/pepper_udp_socket_message_filter.cc
diff --git a/content/browser/renderer_host/pepper/pepper_udp_socket_message_filter.cc b/content/browser/renderer_host/pepper/pepper_udp_socket_message_filter.cc
index 50221d9222c4f7dfe3e7f2c87e3218020c864d48..dd58c40846696bb8ca301c2d252cd591efc811ed 100644
--- a/content/browser/renderer_host/pepper/pepper_udp_socket_message_filter.cc
+++ b/content/browser/renderer_host/pepper/pepper_udp_socket_message_filter.cc
@@ -26,12 +26,14 @@
 #include "ppapi/host/ppapi_host.h"
 #include "ppapi/host/resource_host.h"
 #include "ppapi/proxy/ppapi_messages.h"
+#include "ppapi/proxy/udp_socket_filter.h"
 #include "ppapi/proxy/udp_socket_resource_base.h"
 #include "ppapi/shared_impl/private/net_address_private_impl.h"
 #include "ppapi/shared_impl/socket_option_data.h"
 
 using ppapi::NetAddressPrivateImpl;
 using ppapi::host::NetErrorToPepperError;
+using ppapi::proxy::UDPSocketFilter;
 using ppapi::proxy::UDPSocketResourceBase;
 
 namespace {
@@ -61,7 +63,7 @@ PepperUDPSocketMessageFilter::PepperUDPSocketMessageFilter(
       rcvbuf_size_(0),
       sndbuf_size_(0),
       closed_(false),
-      remaining_recv_slots_(UDPSocketResourceBase::kPluginReceiveBufferSlots),
+      remaining_recv_slots_(UDPSocketFilter::kPluginReceiveBufferSlots),
       external_plugin_(host->external_plugin()),
       private_api_(private_api),
       render_process_id_(0),
@@ -167,8 +169,7 @@ int32_t PepperUDPSocketMessageFilter::OnMsgSetOption(
       int32_t integer_value = 0;
       if (!value.GetInt32(&integer_value) ||
           integer_value <= 0 ||
-          integer_value >
-              ppapi::proxy::UDPSocketResourceBase::kMaxSendBufferSize)
+          integer_value > UDPSocketResourceBase::kMaxSendBufferSize)
         return PP_ERROR_BADARGUMENT;
 
       // If the socket is already connected, proxy the value to UDPSocket.
@@ -186,8 +187,7 @@ int32_t PepperUDPSocketMessageFilter::OnMsgSetOption(
       int32_t integer_value = 0;
       if (!value.GetInt32(&integer_value) ||
           integer_value <= 0 ||
-          integer_value >
-              ppapi::proxy::UDPSocketResourceBase::kMaxReceiveBufferSize)
+          integer_value > UDPSocketFilter::kMaxReceiveBufferSize)
         return PP_ERROR_BADARGUMENT;
 
       // If the socket is already connected, proxy the value to UDPSocket.
@@ -273,8 +273,7 @@ int32_t PepperUDPSocketMessageFilter::OnMsgRecvSlotAvailable(
     const ppapi::host::HostMessageContext* context) {
   DCHECK_CURRENTLY_ON(BrowserThread::IO);
 
-  if (remaining_recv_slots_ <
-      UDPSocketResourceBase::kPluginReceiveBufferSlots) {
+  if (remaining_recv_slots_ < UDPSocketFilter::kPluginReceiveBufferSlots) {
     remaining_recv_slots_++;
   }
 
@@ -381,7 +380,7 @@ void PepperUDPSocketMessageFilter::DoRecvFrom() {
   DCHECK(!recvfrom_buffer_.get());
   DCHECK_GT(remaining_recv_slots_, 0u);
 
-  recvfrom_buffer_ = new net::IOBuffer(UDPSocketResourceBase::kMaxReadSize);
+  recvfrom_buffer_ = new net::IOBuffer(UDPSocketFilter::kMaxReadSize);
 
   // Use base::Unretained(this), so that the lifespan of this object doesn't
   // have to last until the callback is called.
@@ -390,7 +389,7 @@ void PepperUDPSocketMessageFilter::DoRecvFrom() {
   // called.
   int net_result = socket_->RecvFrom(
       recvfrom_buffer_.get(),
-      UDPSocketResourceBase::kMaxReadSize,
+      UDPSocketFilter::kMaxReadSize,
       &recvfrom_address_,
       base::Bind(&PepperUDPSocketMessageFilter::OnRecvFromCompleted,
                  base::Unretained(this)));