diff options
author | ajwong@chromium.org <ajwong@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-12-09 18:43:55 +0000 |
---|---|---|
committer | ajwong@chromium.org <ajwong@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-12-09 18:43:55 +0000 |
commit | 83039bbf2f2ec0e918f7000b5212d104f60f2bb7 (patch) | |
tree | b22dbd0051b57a437a588772a874271f0d02ffdb /jingle/glue/channel_socket_adapter_unittest.cc | |
parent | e7456a206fe5b50aeb322ebabd6c26adc869a5fd (diff) | |
download | chromium_src-83039bbf2f2ec0e918f7000b5212d104f60f2bb7.zip chromium_src-83039bbf2f2ec0e918f7000b5212d104f60f2bb7.tar.gz chromium_src-83039bbf2f2ec0e918f7000b5212d104f60f2bb7.tar.bz2 |
Migrate net/socket/socket.h, net/socket/stream_socket.h to base::Bind().
This changes Socket::Read(), Socket::Write, and StreamSocket::Connect() to use CompletionCallback and fixes all users.
BUG=none
TEST=existing.
Review URL: http://codereview.chromium.org/8824006
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@113825 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'jingle/glue/channel_socket_adapter_unittest.cc')
-rw-r--r-- | jingle/glue/channel_socket_adapter_unittest.cc | 15 |
1 files changed, 8 insertions, 7 deletions
diff --git a/jingle/glue/channel_socket_adapter_unittest.cc b/jingle/glue/channel_socket_adapter_unittest.cc index e8b34b5..7f5f3ee 100644 --- a/jingle/glue/channel_socket_adapter_unittest.cc +++ b/jingle/glue/channel_socket_adapter_unittest.cc @@ -44,7 +44,8 @@ class TransportChannelSocketAdapterTest : public testing::Test { public: TransportChannelSocketAdapterTest() : ALLOW_THIS_IN_INITIALIZER_LIST( - callback_(this, &TransportChannelSocketAdapterTest::Callback)), + callback_(base::Bind(&TransportChannelSocketAdapterTest::Callback, + base::Unretained(this)))), callback_result_(0) { } @@ -59,7 +60,7 @@ class TransportChannelSocketAdapterTest : public testing::Test { MockTransportChannel channel_; scoped_ptr<TransportChannelSocketAdapter> target_; - net::OldCompletionCallbackImpl<TransportChannelSocketAdapterTest> callback_; + net::CompletionCallback callback_; int callback_result_; MessageLoopForIO message_loop_; }; @@ -68,7 +69,7 @@ class TransportChannelSocketAdapterTest : public testing::Test { TEST_F(TransportChannelSocketAdapterTest, Read) { scoped_refptr<IOBuffer> buffer(new IOBuffer(kBufferSize)); - int result = target_->Read(buffer, kBufferSize, &callback_); + int result = target_->Read(buffer, kBufferSize, callback_); ASSERT_EQ(net::ERR_IO_PENDING, result); channel_.SignalReadPacket(&channel_, kTestData, kTestDataSize); @@ -79,14 +80,14 @@ TEST_F(TransportChannelSocketAdapterTest, Read) { TEST_F(TransportChannelSocketAdapterTest, ReadClose) { scoped_refptr<IOBuffer> buffer(new IOBuffer(kBufferSize)); - int result = target_->Read(buffer, kBufferSize, &callback_); + int result = target_->Read(buffer, kBufferSize, callback_); ASSERT_EQ(net::ERR_IO_PENDING, result); target_->Close(kTestError); EXPECT_EQ(kTestError, callback_result_); // All Read() calls after Close() should return the error. - EXPECT_EQ(kTestError, target_->Read(buffer, kBufferSize, &callback_)); + EXPECT_EQ(kTestError, target_->Read(buffer, kBufferSize, callback_)); } // Verify that Write sends the packet and returns correct result. @@ -96,7 +97,7 @@ TEST_F(TransportChannelSocketAdapterTest, Write) { EXPECT_CALL(channel_, SendPacket(buffer->data(), kTestDataSize)) .WillOnce(Return(kTestDataSize)); - int result = target_->Write(buffer, kTestDataSize, &callback_); + int result = target_->Write(buffer, kTestDataSize, callback_); EXPECT_EQ(kTestDataSize, result); } @@ -112,7 +113,7 @@ TEST_F(TransportChannelSocketAdapterTest, WritePending) { EXPECT_CALL(channel_, GetError()) .WillOnce(Return(EWOULDBLOCK)); - int result = target_->Write(buffer, kTestDataSize, &callback_); + int result = target_->Write(buffer, kTestDataSize, callback_); ASSERT_EQ(net::OK, result); } |