RefCounted types should not have public destructors, net/

net/proxy/proxy_script_decider_unittest.cc

 // Copyright (c) 2011 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 <vector>
 
 #include "base/bind.h"
 #include "base/message_loop.h"
 #include "base/string_util.h"
 #include "base/time.h"
@@ skipping 540 matching lines @@
   const GURL& GetPacURL() const OVERRIDE {
     return dummy_gurl_;
   }
 
   void CallbackWithFailure() {
     if (!callback_.is_null())
       callback_.Run(ERR_PAC_NOT_IN_DHCP);
   }
 
  private:
+  friend class base::RefCountedThreadSafe<AsyncFailDhcpFetcher>;
+  ~AsyncFailDhcpFetcher() {}
+
   GURL dummy_gurl_;
   CompletionCallback callback_;
 };
 
 TEST(ProxyScriptDeciderTest, DhcpCancelledByDestructor) {
   // This regression test would crash before
   // http://codereview.chromium.org/7044058/
   // Thus, we don't care much about actual results (hence no EXPECT or ASSERT
   // macros below), just that it doesn't crash.
   Rules rules;
@@ skipping 15 matching lines @@
 
   // Run the message loop to let the DHCP fetch complete and post the results
   // back. Before the fix linked to above, this would try to invoke on
   // the callback object provided by ProxyScriptDecider after it was
   // no longer valid.
   MessageLoop::current()->RunAllPending();
 }
 
 }  // namespace
 }  // namespace net