Remove ClientSocketFactory constructors that take ClientSocketHandles.
All StreamSocket layers are now beneath the socket pools, so there's no
longer any reason for any of them to be on top of ClientSocketHandles
instead of other SocketStreams.
Bug: 472729
Change-Id: I1b0471c421181eee2c28388bc1b705011d0d3dd9
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1495596
Reviewed-by: mark a. foltz <[email protected]>
Reviewed-by: Asanka Herath <[email protected]>
Reviewed-by: Reilly Grant <[email protected]>
Reviewed-by: Sergey Ulanov <[email protected]>
Commit-Queue: Matt Menke <[email protected]>
Cr-Commit-Position: refs/heads/master@{#637887}diff --git a/net/socket/ssl_server_socket_unittest.cc b/net/socket/ssl_server_socket_unittest.cc
index 78bb3ca7..c5d55f6 100644
--- a/net/socket/ssl_server_socket_unittest.cc
+++ b/net/socket/ssl_server_socket_unittest.cc
@@ -417,12 +417,10 @@
server_socket_.reset();
channel_1_.reset(new FakeDataChannel());
channel_2_.reset(new FakeDataChannel());
- std::unique_ptr<ClientSocketHandle> client_connection(
- new ClientSocketHandle);
- client_connection->SetSocket(std::unique_ptr<StreamSocket>(
- new FakeSocket(channel_1_.get(), channel_2_.get())));
- std::unique_ptr<StreamSocket> server_socket(
- new FakeSocket(channel_2_.get(), channel_1_.get()));
+ std::unique_ptr<StreamSocket> client_connection =
+ std::make_unique<FakeSocket>(channel_1_.get(), channel_2_.get());
+ std::unique_ptr<StreamSocket> server_socket =
+ std::make_unique<FakeSocket>(channel_2_.get(), channel_1_.get());
HostPortPair host_and_pair("unittest", 0);
SSLClientSocketContext context(
