diff options
| author | rch@chromium.org <rch@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-09-25 21:09:38 +0000 |
|---|---|---|
| committer | rch@chromium.org <rch@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-09-25 21:09:38 +0000 |
| commit | 02b0c3408a241a812a46b20c1cc361cba5d2bf9b (patch) | |
| tree | ccafc9271ba932fd8596326172755b75f08108fb /net/spdy/spdy_stream_unittest.cc | |
| parent | 76aece4cb1cd2983e4653a0a5afc95bc8496adf5 (diff) | |
| download | chromium_src-02b0c3408a241a812a46b20c1cc361cba5d2bf9b.zip chromium_src-02b0c3408a241a812a46b20c1cc361cba5d2bf9b.tar.gz chromium_src-02b0c3408a241a812a46b20c1cc361cba5d2bf9b.tar.bz2 | |
Remove SpdySession::Connect(). This methods was only
being called from a handful of unit tests. I modified
those tests to call SpdySession::InitializeWithSocket()
instead. This added a bit of code to these tests.
BUG=56848
TEST=none
Review URL: http://codereview.chromium.org/3435029
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@60589 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'net/spdy/spdy_stream_unittest.cc')
| -rw-r--r-- | net/spdy/spdy_stream_unittest.cc | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/net/spdy/spdy_stream_unittest.cc b/net/spdy/spdy_stream_unittest.cc index 392fb3b..54ea82d 100644 --- a/net/spdy/spdy_stream_unittest.cc +++ b/net/spdy/spdy_stream_unittest.cc @@ -192,8 +192,12 @@ TEST_F(SpdyStreamTest, SendDataAfterOpen) { HostPortPair host_port_pair("www.google.com", 80); scoped_refptr<TCPSocketParams> tcp_params = new TCPSocketParams(host_port_pair, LOWEST, GURL(), false); - EXPECT_EQ(OK, session->Connect("spdy.www.google.com", tcp_params, - LOWEST)); + + scoped_ptr<ClientSocketHandle> connection(new ClientSocketHandle); + EXPECT_EQ(OK, + connection->Init(host_port_pair.ToString(), tcp_params, LOWEST, + NULL, session_->tcp_socket_pool(), BoundNetLog())); + session->InitializeWithSocket(connection.release(), false, OK); scoped_refptr<SpdyStream> stream; ASSERT_EQ( |