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

Unified Diff: net/spdy/spdy_session_spdy3_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 | « net/spdy/spdy_session_spdy2_unittest.cc ('k') | net/spdy/spdy_stream_test_util.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/spdy/spdy_session_spdy3_unittest.cc
diff --git a/net/spdy/spdy_session_spdy3_unittest.cc b/net/spdy/spdy_session_spdy3_unittest.cc
index 870e006619b041af31ee0b1d87cdc16f8e58d428..7ce6ded778ddc30c4db988d52b6e29b2ffa4176b 100644
--- a/net/spdy/spdy_session_spdy3_unittest.cc
+++ b/net/spdy/spdy_session_spdy3_unittest.cc
@@ -397,9 +397,9 @@ TEST_F(SpdySessionSpdy3Test, DeleteExpiredPushStreams) {
(*request_headers)[":path"] = "/";
scoped_refptr<SpdyStream> stream(
- new SpdyStream(session, false, session->net_log_));
+ new SpdyStream(session.get(), false, session->net_log_));
stream->set_spdy_headers(request_headers.Pass());
- session->ActivateStream(stream);
+ session->ActivateStream(stream.get());
SpdyHeaderBlock headers;
headers[":scheme"] = "http";
@@ -627,8 +627,8 @@ TEST_F(SpdySessionSpdy3Test, CloseIdleSessions) {
// Make sessions 1 and 3 inactive, but keep them open.
// Session 2 still open and active
- session1->CloseCreatedStream(spdy_stream1, OK);
- session3->CloseCreatedStream(spdy_stream3, OK);
+ session1->CloseCreatedStream(spdy_stream1.get(), OK);
+ session3->CloseCreatedStream(spdy_stream3.get(), OK);
EXPECT_FALSE(session1->is_active());
EXPECT_FALSE(session1->IsClosed());
EXPECT_TRUE(session2->is_active());
@@ -651,7 +651,7 @@ TEST_F(SpdySessionSpdy3Test, CloseIdleSessions) {
EXPECT_FALSE(session2->IsClosed());
// Make 2 not active
- session2->CloseCreatedStream(spdy_stream2, OK);
+ session2->CloseCreatedStream(spdy_stream2.get(), OK);
EXPECT_FALSE(session2->is_active());
EXPECT_FALSE(session2->IsClosed());
@@ -739,7 +739,7 @@ TEST_F(SpdySessionSpdy3Test, OnSettings) {
BoundNetLog(),
callback1.callback()));
- StreamReleaserCallback stream_releaser(session, spdy_stream1);
+ StreamReleaserCallback stream_releaser(session.get(), spdy_stream1.get());
ASSERT_EQ(ERR_IO_PENDING,
session->CreateStream(url,
« no previous file with comments | « net/spdy/spdy_session_spdy2_unittest.cc ('k') | net/spdy/spdy_stream_test_util.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698