Revert 132842 - If we are using blocking write, when the renderer stop getting the data without not…

base/sync_socket.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef BASE_SYNC_SOCKET_H_
 #define BASE_SYNC_SOCKET_H_
 #pragma once
 
 // A socket abstraction used for sending and receiving plain
-// data.  Because the receiving is blocking, they can be used to perform
+// data.  Because they are blocking, they can be used to perform
 // rudimentary cross-process synchronization with low latency.
 
 #include "base/basictypes.h"
 #if defined(OS_WIN)
 #include <windows.h>
 #endif
 #include <sys/types.h>
 
 #include "base/base_export.h"
 #include "base/compiler_specific.h"
@@ skipping 49 matching lines @@
   Handle handle() const { return handle_; }
 
  protected:
   Handle handle_;
 
  private:
   DISALLOW_COPY_AND_ASSIGN(SyncSocket);
 };
 
 // Derives from SyncSocket and adds support for shutting down the socket from
-// another thread while a blocking Receive or Send is being done from the
-// thread that owns the socket.
+// another thread while a blocking Receive or Send is being done from the thread
+// that owns the socket.
 class BASE_EXPORT CancelableSyncSocket : public SyncSocket {
  public:
   CancelableSyncSocket();
   explicit CancelableSyncSocket(Handle handle);
   virtual ~CancelableSyncSocket() {}
 
   // Initializes a pair of cancelable sockets.  See documentation for
   // SyncSocket::CreatePair for more details.
   static bool CreatePair(CancelableSyncSocket* socket_a,
                          CancelableSyncSocket* socket_b);
 
   // A way to shut down a socket even if another thread is currently performing
   // a blocking Receive or Send.
   bool Shutdown();
 
 #if defined(OS_WIN)
   // Since the Linux and Mac implementations actually use a socket, shutting
   // them down from another thread is pretty simple - we can just call
   // shutdown().  However, the Windows implementation relies on named pipes
   // and there isn't a way to cancel a blocking synchronous Read that is
   // supported on <Vista. So, for Windows only, we override these
   // SyncSocket methods in order to support shutting down the 'socket'.
   virtual bool Close() OVERRIDE;
+  virtual size_t Send(const void* buffer, size_t length) OVERRIDE;
   virtual size_t Receive(void* buffer, size_t length) OVERRIDE;
 #endif
 
-  // Send() is overridden to catch cases where the remote end is not responding
-  // and we fill the local socket buffer. When the buffer is full, this
-  // implementation of Send() will not block indefinitely as
-  // SyncSocket::Send will, but instead return 0, as no bytes could be sent.
-  // Note that the socket will not be closed in this case.
-  virtual size_t Send(const void* buffer, size_t length) OVERRIDE;
-
  private:
 #if defined(OS_WIN)
   WaitableEvent shutdown_event_;
   WaitableEvent file_operation_;
 #endif
   DISALLOW_COPY_AND_ASSIGN(CancelableSyncSocket);
 };
 
 }  // namespace base
 
 #endif  // BASE_SYNC_SOCKET_H_