Pass ClientSocketHandle ownership around in unique_ptr.

net/http/http_basic_stream.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "net/http/http_basic_stream.h"
 
-#include <memory>
+#include <utility>
 
 #include "net/http/http_request_info.h"
 #include "net/http/http_response_body_drainer.h"
 #include "net/http/http_stream_parser.h"
 #include "net/socket/client_socket_handle.h"
 
 namespace net {
 
-HttpBasicStream::HttpBasicStream(ClientSocketHandle* connection,
+HttpBasicStream::HttpBasicStream(std::unique_ptr<ClientSocketHandle> connection,
                                  bool using_proxy)
-    : state_(connection, using_proxy) {}
+    : state_(std::move(connection), using_proxy) {}
 
 HttpBasicStream::~HttpBasicStream() {}
 
 int HttpBasicStream::InitializeStream(const HttpRequestInfo* request_info,
                                       RequestPriority priority,
                                       const BoundNetLog& net_log,
                                       const CompletionCallback& callback) {
   state_.Initialize(request_info, priority, net_log, callback);
   return OK;
 }
@@ skipping 24 matching lines @@
   parser()->Close(not_reusable);
 }
 
 HttpStream* HttpBasicStream::RenewStreamForAuth() {
   DCHECK(IsResponseBodyComplete());
   DCHECK(!parser()->IsMoreDataBuffered());
   // The HttpStreamParser object still has a pointer to the connection. Just to
   // be extra-sure it doesn't touch the connection again, delete it here rather
   // than leaving it until the destructor is called.
   state_.DeleteParser();
-  return new HttpBasicStream(state_.ReleaseConnection().release(),
-                             state_.using_proxy());
+  return new HttpBasicStream(state_.ReleaseConnection(), state_.using_proxy());
 }
 
 bool HttpBasicStream::IsResponseBodyComplete() const {
   return parser()->IsResponseBodyComplete();
 }
 
 bool HttpBasicStream::IsConnectionReused() const {
   return parser()->IsConnectionReused();
 }
 
@@ skipping 53 matching lines @@
   // like the right version should be reported, if headers were received.
   details->connection_info = HttpResponseInfo::CONNECTION_INFO_HTTP1_1;
   return;
 }
 
 void HttpBasicStream::SetPriority(RequestPriority priority) {
   // TODO(akalin): Plumb this through to |connection_|.
 }
 
 }  // namespace net