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

Unified Diff: net/websockets/websocket_basic_stream_test.cc

Issue 474973004: Remove implicit conversions from scoped_refptr to T* in net/websockets/ (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 4 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
Index: net/websockets/websocket_basic_stream_test.cc
diff --git a/net/websockets/websocket_basic_stream_test.cc b/net/websockets/websocket_basic_stream_test.cc
index 71af0797818b41c21ca84f86d115a179bb51973b..c67583d2dc91c79df8a6afacce8280cc200250a9 100644
--- a/net/websockets/websocket_basic_stream_test.cc
+++ b/net/websockets/websocket_basic_stream_test.cc
@@ -639,7 +639,7 @@ TEST_F(WebSocketBasicStreamSocketTest, HttpReadBufferIsUsed) {
EXPECT_EQ(OK, stream_->ReadFrames(&frames_, cb_.callback()));
ASSERT_EQ(1U, frames_.size());
- ASSERT_TRUE(frames_[0]->data);
+ ASSERT_TRUE(frames_[0]->data.get());
EXPECT_EQ(GG_UINT64_C(6), frames_[0]->header.payload_length);
}
@@ -653,7 +653,7 @@ TEST_F(WebSocketBasicStreamSocketSingleReadTest,
ASSERT_EQ(ERR_IO_PENDING, stream_->ReadFrames(&frames_, cb_.callback()));
EXPECT_EQ(OK, cb_.WaitForResult());
ASSERT_EQ(1U, frames_.size());
- ASSERT_TRUE(frames_[0]->data);
+ ASSERT_TRUE(frames_[0]->data.get());
EXPECT_EQ(GG_UINT64_C(6), frames_[0]->header.payload_length);
EXPECT_EQ(WebSocketFrameHeader::kOpCodeText, frames_[0]->header.opcode);
}

Powered by Google App Engine
This is Rietveld 408576698