RefCounted types should not have public destructors, net/

net/cookies/cookie_monster_store_test.h

Index: net/cookies/cookie_monster_store_test.h
diff --git a/net/cookies/cookie_monster_store_test.h b/net/cookies/cookie_monster_store_test.h
index a05ddd3b5538a4b14bddc5cb9437869223327e2a..cc5d566f696154d827b78c6a4d675a86727721c6 100644
--- a/net/cookies/cookie_monster_store_test.h
+++ b/net/cookies/cookie_monster_store_test.h
@@ -33,13 +33,15 @@ class LoadedCallbackTask
 
   LoadedCallbackTask(LoadedCallback loaded_callback,
                      std::vector<CookieMonster::CanonicalCookie*> cookies);
-  ~LoadedCallbackTask();
 
   void Run() {
     loaded_callback_.Run(cookies_);
   }
 
  private:
+  friend class base::RefCountedThreadSafe<LoadedCallbackTask>;
+  ~LoadedCallbackTask();
+
   LoadedCallback loaded_callback_;
   std::vector<CookieMonster::CanonicalCookie*> cookies_;
 
@@ -72,7 +74,6 @@ class MockPersistentCookieStore
   typedef std::vector<CookieStoreCommand> CommandList;
 
   MockPersistentCookieStore();
-  virtual ~MockPersistentCookieStore();
 
   void SetLoadExpectation(
       bool return_value,
@@ -101,6 +102,8 @@ class MockPersistentCookieStore
   virtual void SetClearLocalStateOnExit(bool clear_local_state) OVERRIDE;
 
  private:
+  virtual ~MockPersistentCookieStore();
+
   CommandList commands_;
 
   // Deferred result to use when Load() is called.
@@ -157,7 +160,6 @@ class MockSimplePersistentCookieStore
     : public CookieMonster::PersistentCookieStore {
  public:
   MockSimplePersistentCookieStore();
-  virtual ~MockSimplePersistentCookieStore();
 
   virtual void Load(const LoadedCallback& loaded_callback) OVERRIDE;
 
@@ -178,6 +180,8 @@ class MockSimplePersistentCookieStore
   virtual void SetClearLocalStateOnExit(bool clear_local_state) OVERRIDE;
 
  private:
+  virtual ~MockSimplePersistentCookieStore();
+
   typedef std::map<int64, CookieMonster::CanonicalCookie>
       CanonicalCookieMap;