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

Unified Diff: net/spdy/spdy_session_spdy2_unittest.cc

Issue 15829004: Update net/ to use scoped_refptr<T>::get() rather than implicit "operator T*" (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: license twerk Created 7 years, 7 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_pool.cc ('k') | net/spdy/spdy_session_spdy3_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/spdy/spdy_session_spdy2_unittest.cc
diff --git a/net/spdy/spdy_session_spdy2_unittest.cc b/net/spdy/spdy_session_spdy2_unittest.cc
index fded521b73086544c4d252c2fb3481f1bd769dc0..aad6da45c50244aed2bb40fd898f17db669ed808 100644
--- a/net/spdy/spdy_session_spdy2_unittest.cc
+++ b/net/spdy/spdy_session_spdy2_unittest.cc
@@ -336,12 +336,13 @@ TEST_F(SpdySessionSpdy2Test, DeleteExpiredPushStreams) {
(*request_headers)["host"] = "www.google.com";
(*request_headers)["url"] = "/";
- scoped_ptr<SpdyStream> stream(
- new SpdyStream(SPDY_REQUEST_RESPONSE_STREAM,
- session, std::string(), DEFAULT_PRIORITY,
- kSpdyStreamInitialWindowSize,
- kSpdyStreamInitialWindowSize,
- session->net_log_));
+ scoped_ptr<SpdyStream> stream(new SpdyStream(SPDY_REQUEST_RESPONSE_STREAM,
+ session.get(),
+ std::string(),
+ DEFAULT_PRIORITY,
+ kSpdyStreamInitialWindowSize,
+ kSpdyStreamInitialWindowSize,
+ session->net_log_));
stream->SendRequestHeaders(request_headers.Pass(), NO_MORE_DATA_TO_SEND);
SpdyStream* stream_ptr = stream.get();
session->InsertCreatedStream(stream.Pass());
@@ -929,7 +930,9 @@ void IPPoolingTest(SpdyPoolCloseSessionsType close_sessions_type) {
false,
OnHostResolutionCallback()));
IPPoolingInitializedSession(test_host_port_pair.ToString(),
- transport_params, http_session, session);
+ transport_params,
+ http_session.get(),
+ session.get());
// TODO(rtenneti): MockClientSocket::GetPeerAddress return's 0 as the port
// number. Fix it to return port 80 and then use GetPeerAddress to AddAlias.
@@ -966,7 +969,9 @@ void IPPoolingTest(SpdyPoolCloseSessionsType close_sessions_type) {
// Initialize session for host 2.
session_deps.socket_factory->AddSocketDataProvider(&data);
IPPoolingInitializedSession(test_hosts[2].key.host_port_pair().ToString(),
- transport_params, http_session, session2);
+ transport_params,
+ http_session.get(),
+ session2.get());
// Grab the session to host 1 and verify that it is the same session
// we got with host 0, and that is a different than host 2's session.
@@ -978,7 +983,9 @@ void IPPoolingTest(SpdyPoolCloseSessionsType close_sessions_type) {
// Initialize session for host 1.
session_deps.socket_factory->AddSocketDataProvider(&data);
IPPoolingInitializedSession(test_hosts[2].key.host_port_pair().ToString(),
- transport_params, http_session, session2);
+ transport_params,
+ http_session.get(),
+ session2.get());
// Remove the aliases and observe that we still have a session for host1.
pool_peer.RemoveAliases(test_hosts[0].key);
« no previous file with comments | « net/spdy/spdy_session_pool.cc ('k') | net/spdy/spdy_session_spdy3_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698