diff options
author | mbelshe@google.com <mbelshe@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-11-11 01:41:59 +0000 |
---|---|---|
committer | mbelshe@google.com <mbelshe@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-11-11 01:41:59 +0000 |
commit | 5ecc992a4bf8546b21db89901f2d87e8b4ae1a94 (patch) | |
tree | c96fe65384323dee8a3d7c69dec9ae41ffe96801 /net/http/http_network_layer_unittest.cc | |
parent | d871f2ede5f2e511fdb621305ceef207f1c9e52d (diff) | |
download | chromium_src-5ecc992a4bf8546b21db89901f2d87e8b4ae1a94.zip chromium_src-5ecc992a4bf8546b21db89901f2d87e8b4ae1a94.tar.gz chromium_src-5ecc992a4bf8546b21db89901f2d87e8b4ae1a94.tar.bz2 |
There was confusion in the mock socket classes due to
overlapping names. The MockSocket is not actually a socket.
It is an interface for fetching data for reads and writes
on a socket. The MockClientSocket and MockTCPClientSocket
are the actual sockets.
Rename MockSocket to SocketDataProvider.
Rename SSLMockSocket to SSLSocketDataProvider.
Update all tests to reflect name change.
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/384024
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@31640 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'net/http/http_network_layer_unittest.cc')
-rw-r--r-- | net/http/http_network_layer_unittest.cc | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/net/http/http_network_layer_unittest.cc b/net/http/http_network_layer_unittest.cc index dcc9405..d8240e1 100644 --- a/net/http/http_network_layer_unittest.cc +++ b/net/http/http_network_layer_unittest.cc @@ -62,8 +62,8 @@ TEST_F(HttpNetworkLayerTest, GET) { "Connection: keep-alive\r\n" "User-Agent: Foo/1.0\r\n\r\n"), }; - net::StaticMockSocket data(data_reads, data_writes); - mock_socket_factory.AddMockSocket(&data); + net::StaticSocketDataProvider data(data_reads, data_writes); + mock_socket_factory.AddSocketDataProvider(&data); net::HttpNetworkLayer factory(&mock_socket_factory, new net::MockHostResolver, net::ProxyService::CreateNull(), |