DeterministicSocketDataProvider should not be RefCounted

net/http/http_pipelined_connection_impl_unittest.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_pipelined_connection_impl.h"
 
 #include <string>
 
 #include "base/memory/ref_counted.h"
+#include "base/memory/scoped_ptr.h"
 #include "base/memory/scoped_vector.h"
 #include "net/base/io_buffer.h"
 #include "net/base/net_errors.h"
 #include "net/base/request_priority.h"
 #include "net/http/http_pipelined_stream.h"
 #include "net/socket/client_socket_handle.h"
 #include "net/socket/client_socket_pool_histograms.h"
 #include "net/socket/socket_test_util.h"
 #include "testing/gmock/include/gmock/gmock.h"
 #include "testing/gtest/include/gtest/gtest.h"
@@ skipping 53 matching lines @@
         pool_(1, 1, &histograms_, &factory_),
         origin_("host", 123) {
   }
 
   void TearDown() {
     MessageLoop::current()->RunAllPending();
   }
 
   void Initialize(MockRead* reads, size_t reads_count,
                   MockWrite* writes, size_t writes_count) {
-    data_ = new DeterministicSocketData(reads, reads_count,
-                                        writes, writes_count);
+    data_.reset(new DeterministicSocketData(reads, reads_count,
+                                            writes, writes_count));
     data_->set_connect_data(MockConnect(SYNCHRONOUS, OK));
     if (reads_count || writes_count) {
       data_->StopAfter(reads_count + writes_count);
     }
     factory_.AddSocketDataProvider(data_.get());
     scoped_refptr<DummySocketParams> params;
     ClientSocketHandle* connection = new ClientSocketHandle;
     connection->Init("a", params, MEDIUM, CompletionCallback(), &pool_,
                      BoundNetLog());
     pipeline_.reset(new HttpPipelinedConnectionImpl(
@@ skipping 46 matching lines @@
         callback_.callback()));
     EXPECT_EQ(OK, stream->ReadResponseHeaders(callback_.callback()));
     ExpectResponse(filename, stream, false);
 
     stream->Close(false);
   }
 
   DeterministicMockClientSocketFactory factory_;
   ClientSocketPoolHistograms histograms_;
   MockTransportClientSocketPool pool_;
-  scoped_refptr<DeterministicSocketData> data_;
+  scoped_ptr<DeterministicSocketData> data_;
 
   HostPortPair origin_;
   SSLConfig ssl_config_;
   ProxyInfo proxy_info_;
   NiceMock<MockPipelineDelegate> delegate_;
   TestCompletionCallback callback_;
   scoped_ptr<HttpPipelinedConnectionImpl> pipeline_;
   ScopedVector<HttpRequestInfo> request_info_vector_;
 };
 
@@ skipping 1440 matching lines @@
   MessageLoop::current()->RunAllPending();
 
   stream->Close(false);
   EXPECT_CALL(delegate_, OnPipelineHasCapacity(pipeline_.get())).Times(1);
   stream.reset(NULL);
 }
 
 }  // anonymous namespace
 
 }  // namespace net