Simplify BufferedSocketWriter by making it non-thread-safe.

Simplify BufferedSocketWriter by making it non-thread-safe.

Originally BufferedSocketWriter was implemented to be thread-safe,
so that any thread can call Write() method. We don't write any data from
other threads anymore, so there is no need to keep it thread-safe.

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=138694

[Alpha Left Google, 2012-05-23 20:28:37]

LGTM with 2 nits.

http://codereview.chromium.org/10411082/diff/2001/remoting/protocol/buffered_...
File remoting/protocol/buffered_socket_writer.h (right):

http://codereview.chromium.org/10411082/diff/2001/remoting/protocol/buffered_...
remoting/protocol/buffered_socket_writer.h:36: explicit
BufferedSocketWriterBase();
No need to be explicit anymore.

http://codereview.chromium.org/10411082/diff/2001/remoting/protocol/buffered_...
remoting/protocol/buffered_socket_writer.h:98: explicit BufferedSocketWriter();
Same here.

[Sergey Ulanov, 2012-05-23 22:29:00]

http://codereview.chromium.org/10411082/diff/2001/remoting/protocol/buffered_...
File remoting/protocol/buffered_socket_writer.h (right):

http://codereview.chromium.org/10411082/diff/2001/remoting/protocol/buffered_...
remoting/protocol/buffered_socket_writer.h:36: explicit
BufferedSocketWriterBase();
On 2012/05/23 20:28:37, Alpha wrote:
> No need to be explicit anymore.

Done.

http://codereview.chromium.org/10411082/diff/2001/remoting/protocol/buffered_...
remoting/protocol/buffered_socket_writer.h:98: explicit BufferedSocketWriter();
On 2012/05/23 20:28:37, Alpha wrote:
> Same here.

Done.

[commit-bot: I haz the power, 2012-05-23 22:30:37]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/[email protected]/10411082/7001