RefCounted types should not have public destructors

chrome/browser/password_manager/password_store_unittest.cc

Index: chrome/browser/password_manager/password_store_unittest.cc
diff --git a/chrome/browser/password_manager/password_store_unittest.cc b/chrome/browser/password_manager/password_store_unittest.cc
index 0eb4149eeb32e55c24c94c99165646b85da08d99..a15fbf8fc7fea8b5c6f313999925ea79f9117984 100644
--- a/chrome/browser/password_manager/password_store_unittest.cc
+++ b/chrome/browser/password_manager/password_store_unittest.cc
@@ -56,17 +56,15 @@ class DBThreadObserverHelper
     done_event_.Wait();
   }
 
-  virtual ~DBThreadObserverHelper() {
-    DCHECK(BrowserThread::CurrentlyOn(BrowserThread::DB));
-    registrar_.RemoveAll();
-  }
-
   content::NotificationObserverMock& observer() {
     return observer_;
   }
 
  protected:
-  friend class base::RefCountedThreadSafe<DBThreadObserverHelper>;
+  virtual ~DBThreadObserverHelper() {
+    DCHECK(BrowserThread::CurrentlyOn(BrowserThread::DB));
+    registrar_.RemoveAll();
+  }
 
   void AddObserverTask(PasswordStore* password_store) {
     DCHECK(BrowserThread::CurrentlyOn(BrowserThread::DB));
@@ -79,6 +77,10 @@ class DBThreadObserverHelper
   WaitableEvent done_event_;
   content::NotificationRegistrar registrar_;
   content::NotificationObserverMock observer_;
+
+ private:
+  friend struct BrowserThread::DeleteOnThread<BrowserThread::DB>;
+  friend class base::DeleteHelper<DBThreadObserverHelper>;
 };
 
 }  // anonymous namespace