RefCounted types should not have public destructors, net/

net/socket/socket_test_util.h

Index: net/socket/socket_test_util.h
diff --git a/net/socket/socket_test_util.h b/net/socket/socket_test_util.h
index 350e0b4439e2dc2c6c4e7d2689d6de2434b0db40..5ec195936eac1aea6c774c4978c1c9442e8ae4ee 100644
--- a/net/socket/socket_test_util.h
+++ b/net/socket/socket_test_util.h
@@ -433,14 +433,14 @@ class DeterministicMockTCPClientSocket;
 //
 // For examples of how to use this class, see:
 //   deterministic_socket_data_unittests.cc
-class DeterministicSocketData : public StaticSocketDataProvider,
-    public base::RefCounted<DeterministicSocketData> {
+class DeterministicSocketData
+    : public StaticSocketDataProvider,
+      public base::RefCounted<DeterministicSocketData> {
  public:
   // |reads| the list of MockRead completions.
   // |writes| the list of MockWrite completions.
   DeterministicSocketData(MockRead* reads, size_t reads_count,
                           MockWrite* writes, size_t writes_count);
-  virtual ~DeterministicSocketData();
 
   // Consume all the data up to the give stop point (via SetStop()).
   void Run();
@@ -476,6 +476,9 @@ class DeterministicSocketData : public StaticSocketDataProvider,
   virtual void CompleteRead() OVERRIDE {}
 
  private:
+  friend class base::RefCounted<DeterministicSocketData>;
+  virtual ~DeterministicSocketData();
+
   // Invoke the read and write callbacks, if the timing is appropriate.
   void InvokeCallbacks();