diff options
author | jhawkins@chromium.org <jhawkins@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-12-09 01:06:19 +0000 |
---|---|---|
committer | jhawkins@chromium.org <jhawkins@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-12-09 01:06:19 +0000 |
commit | f749f9cee070a7a15107fa9cce6b89ad1d969bcf (patch) | |
tree | 0ecb3a7dd6daff154a8e2bf441ee097704651815 /remoting/jingle_glue | |
parent | 4df3c1c675ded79c6c5d2f210a3131808ac93abd (diff) | |
download | chromium_src-f749f9cee070a7a15107fa9cce6b89ad1d969bcf.zip chromium_src-f749f9cee070a7a15107fa9cce6b89ad1d969bcf.tar.gz chromium_src-f749f9cee070a7a15107fa9cce6b89ad1d969bcf.tar.bz2 |
Revert 113699 - base::Bind: Convert Socket::Write.
BUG=none
TEST=none
R=csilv
Review URL: http://codereview.chromium.org/8831001
TBR=jhawkins@chromium.org
Review URL: http://codereview.chromium.org/8889036
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@113719 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'remoting/jingle_glue')
-rw-r--r-- | remoting/jingle_glue/ssl_socket_adapter.cc | 36 | ||||
-rw-r--r-- | remoting/jingle_glue/ssl_socket_adapter.h | 5 |
2 files changed, 7 insertions, 34 deletions
diff --git a/remoting/jingle_glue/ssl_socket_adapter.cc b/remoting/jingle_glue/ssl_socket_adapter.cc index 3df508e..7414c4b 100644 --- a/remoting/jingle_glue/ssl_socket_adapter.cc +++ b/remoting/jingle_glue/ssl_socket_adapter.cc @@ -188,7 +188,7 @@ void SSLSocketAdapter::OnConnectEvent(talk_base::AsyncSocket* socket) { TransportSocket::TransportSocket(talk_base::AsyncSocket* socket, SSLSocketAdapter *ssl_adapter) : old_read_callback_(NULL), - old_write_callback_(NULL), + write_callback_(NULL), read_buffer_len_(0), write_buffer_len_(0), socket_(socket), @@ -327,25 +327,7 @@ int TransportSocket::Read(net::IOBuffer* buf, int buf_len, int TransportSocket::Write(net::IOBuffer* buf, int buf_len, net::OldCompletionCallback* callback) { DCHECK(buf); - DCHECK(!old_write_callback_ && write_callback_.is_null()); - DCHECK(!write_buffer_.get()); - int result = socket_->Send(buf->data(), buf_len); - if (result < 0) { - result = net::MapSystemError(socket_->GetError()); - if (result == net::ERR_IO_PENDING) { - old_write_callback_ = callback; - write_buffer_ = buf; - write_buffer_len_ = buf_len; - } - } - if (result != net::ERR_IO_PENDING) - was_used_to_convey_data_ = true; - return result; -} -int TransportSocket::Write(net::IOBuffer* buf, int buf_len, - const net::CompletionCallback& callback) { - DCHECK(buf); - DCHECK(!old_write_callback_ && write_callback_.is_null()); + DCHECK(!write_callback_); DCHECK(!write_buffer_.get()); int result = socket_->Send(buf->data(), buf_len); if (result < 0) { @@ -404,15 +386,13 @@ void TransportSocket::OnReadEvent(talk_base::AsyncSocket* socket) { } void TransportSocket::OnWriteEvent(talk_base::AsyncSocket* socket) { - if (old_write_callback_ || !write_callback_.is_null()) { + if (write_callback_) { DCHECK(write_buffer_.get()); - net::OldCompletionCallback* old_callback = old_write_callback_; - net::CompletionCallback callback = write_callback_; + net::OldCompletionCallback* callback = write_callback_; scoped_refptr<net::IOBuffer> buffer = write_buffer_; int buffer_len = write_buffer_len_; - old_write_callback_ = NULL; - write_callback_.Reset(); + write_callback_ = NULL; write_buffer_ = NULL; write_buffer_len_ = 0; @@ -420,7 +400,6 @@ void TransportSocket::OnWriteEvent(talk_base::AsyncSocket* socket) { if (result < 0) { result = net::MapSystemError(socket_->GetError()); if (result == net::ERR_IO_PENDING) { - old_write_callback_ = old_callback; write_callback_ = callback; write_buffer_ = buffer; write_buffer_len_ = buffer_len; @@ -428,10 +407,7 @@ void TransportSocket::OnWriteEvent(talk_base::AsyncSocket* socket) { } } was_used_to_convey_data_ = true; - if (old_callback) - old_callback->RunWithParams(Tuple1<int>(result)); - else - callback.Run(result); + callback->RunWithParams(Tuple1<int>(result)); } } diff --git a/remoting/jingle_glue/ssl_socket_adapter.h b/remoting/jingle_glue/ssl_socket_adapter.h index 3638ae1..f929bf8 100644 --- a/remoting/jingle_glue/ssl_socket_adapter.h +++ b/remoting/jingle_glue/ssl_socket_adapter.h @@ -62,8 +62,6 @@ class TransportSocket : public net::StreamSocket, public sigslot::has_slots<> { const net::CompletionCallback& callback) OVERRIDE; virtual int Write(net::IOBuffer* buf, int buf_len, net::OldCompletionCallback* callback) OVERRIDE; - virtual int Write(net::IOBuffer* buf, int buf_len, - const net::CompletionCallback& callback) OVERRIDE; virtual bool SetReceiveBufferSize(int32 size) OVERRIDE; virtual bool SetSendBufferSize(int32 size) OVERRIDE; @@ -75,8 +73,7 @@ class TransportSocket : public net::StreamSocket, public sigslot::has_slots<> { net::OldCompletionCallback* old_read_callback_; net::CompletionCallback read_callback_; - net::OldCompletionCallback* old_write_callback_; - net::CompletionCallback write_callback_; + net::OldCompletionCallback* write_callback_; scoped_refptr<net::IOBuffer> read_buffer_; int read_buffer_len_; |