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

Revert 132842 - If we are using blocking write, when the renderer stop getting the data without notifying the browser, it will hang the browser. This happens with some plugins which use the sync sockets provided by the Pepper.
This patch change CancellableSyncSocket to be non-blocking on sending, so that we don't need to worry the whole browser hangs by one plugin application.

Also, we remove the lock in audio_sync_reader.cc since it is not really needed if we don't set the socket_ to NULL when calling Close(). By doing this we allow the user to close the socket while another thread is writing to the socket.


BUG=121152
TEST=ipc_tests

Review URL: http://codereview.chromium.org/10000004

TBR=xians@chromium.org
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=132888

[Lei Zhang, 2012-04-18 23:05:22]

You got the email already. This CL caused TSAN failures:

http://build.chromium.org/p/chromium.memory.fyi/builders/Chromium%20Mac%20%28...
http://build.chromium.org/p/chromium.memory.fyi/builders/Windows%20Tests%20%2...
(Linux TSAN is still running, but presumably it will also fail)