Chromium Code Reviews| Index: net/quic/quic_client_session_test.cc |
| diff --git a/net/quic/quic_client_session_test.cc b/net/quic/quic_client_session_test.cc |
| index e811648f57a844153fff713f5a2fa443e5a710bb..4d47731e36d1a626c8588b2b04747080a12992fc 100644 |
| --- a/net/quic/quic_client_session_test.cc |
| +++ b/net/quic/quic_client_session_test.cc |
| @@ -60,17 +60,14 @@ TEST_F(QuicClientSessionTest, MaxNumConnections) { |
| std::vector<QuicReliableClientStream*> streams; |
| for (size_t i = 0; i < kDefaultMaxStreamsPerConnection; i++) { |
| QuicReliableClientStream* stream = session_.CreateOutgoingReliableStream(); |
| - streams.push_back(stream); |
| EXPECT_TRUE(stream); |
| + streams.push_back(stream); |
| } |
| EXPECT_FALSE(session_.CreateOutgoingReliableStream()); |
| // Close a stream and ensure I can now open a new one. |
| session_.CloseStream(streams[0]->id()); |
| - scoped_ptr<QuicReliableClientStream> stream( |
| - session_.CreateOutgoingReliableStream()); |
| - EXPECT_TRUE(stream.get()); |
| - STLDeleteElements(&streams); |
| + EXPECT_TRUE(session_.CreateOutgoingReliableStream()); |
|
willchan no longer on Chromium
2012/11/21 22:50:03
Is this just leaking?
Ryan Hamilton
2012/11/21 23:06:35
No, the stream is owned by the session.
|
| } |
| } // namespace |