Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1187)

Unified Diff: remoting/protocol/channel_multiplexer_unittest.cc

Issue 11275088: Remove implicit scoped_refptr operator T* Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 8 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « remoting/protocol/channel_multiplexer.cc ('k') | remoting/protocol/message_decoder.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: remoting/protocol/channel_multiplexer_unittest.cc
diff --git a/remoting/protocol/channel_multiplexer_unittest.cc b/remoting/protocol/channel_multiplexer_unittest.cc
index 781e1180c26cb94fe37d5c9350308663ca04b3a6..7f2812248fc6bd706d2031612f0ca0eda5e29035 100644
--- a/remoting/protocol/channel_multiplexer_unittest.cc
+++ b/remoting/protocol/channel_multiplexer_unittest.cc
@@ -255,10 +255,12 @@ TEST_F(ChannelMultiplexerTest, WriteFailSync) {
EXPECT_CALL(cb2, OnDone(_))
.Times(0);
- EXPECT_EQ(net::ERR_FAILED, host_socket1_->Write(buf, buf->size(), base::Bind(
- &MockSocketCallback::OnDone, base::Unretained(&cb1))));
- EXPECT_EQ(net::ERR_FAILED, host_socket2_->Write(buf, buf->size(), base::Bind(
- &MockSocketCallback::OnDone, base::Unretained(&cb2))));
+ EXPECT_EQ(
+ net::ERR_FAILED, host_socket1_->Write(buf.get(), buf->size(), base::Bind(
+ &MockSocketCallback::OnDone, base::Unretained(&cb1))));
+ EXPECT_EQ(
+ net::ERR_FAILED, host_socket2_->Write(buf.get(), buf->size(), base::Bind(
+ &MockSocketCallback::OnDone, base::Unretained(&cb2))));
message_loop_.RunAllPending();
}
@@ -285,10 +287,10 @@ TEST_F(ChannelMultiplexerTest, WriteFailAsync) {
EXPECT_CALL(cb2, OnDone(net::ERR_FAILED));
EXPECT_EQ(net::ERR_IO_PENDING,
- host_socket1_->Write(buf, buf->size(), base::Bind(
+ host_socket1_->Write(buf.get(), buf->size(), base::Bind(
&MockSocketCallback::OnDone, base::Unretained(&cb1))));
EXPECT_EQ(net::ERR_IO_PENDING,
- host_socket2_->Write(buf, buf->size(), base::Bind(
+ host_socket2_->Write(buf.get(), buf->size(), base::Bind(
&MockSocketCallback::OnDone, base::Unretained(&cb2))));
message_loop_.RunAllPending();
@@ -320,10 +322,10 @@ TEST_F(ChannelMultiplexerTest, DeleteWhenFailed) {
.WillOnce(InvokeWithoutArgs(this, &ChannelMultiplexerTest::DeleteAll));
EXPECT_EQ(net::ERR_IO_PENDING,
- host_socket1_->Write(buf, buf->size(), base::Bind(
+ host_socket1_->Write(buf.get(), buf->size(), base::Bind(
&MockSocketCallback::OnDone, base::Unretained(&cb1))));
EXPECT_EQ(net::ERR_IO_PENDING,
- host_socket2_->Write(buf, buf->size(), base::Bind(
+ host_socket2_->Write(buf.get(), buf->size(), base::Bind(
&MockSocketCallback::OnDone, base::Unretained(&cb2))));
message_loop_.RunAllPending();
« no previous file with comments | « remoting/protocol/channel_multiplexer.cc ('k') | remoting/protocol/message_decoder.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698