RefCounted types should not have public destructors, net/

net/base/default_server_bound_cert_store.h

Index: net/base/default_server_bound_cert_store.h
diff --git a/net/base/default_server_bound_cert_store.h b/net/base/default_server_bound_cert_store.h
index cd664fe7d4d0b9e4f88b1c21e7743c5b64b47c33..daec007bc2808fb4c21deb3b4daf6f4d57155e50 100644
--- a/net/base/default_server_bound_cert_store.h
+++ b/net/base/default_server_bound_cert_store.h
@@ -129,8 +129,6 @@ typedef base::RefCountedThreadSafe<DefaultServerBoundCertStore::PersistentStore>
 class NET_EXPORT DefaultServerBoundCertStore::PersistentStore
     : public RefcountedPersistentStore {
  public:
-  virtual ~PersistentStore() {}
-
   // Initializes the store and retrieves the existing certs. This will be
   // called only once at startup. Note that the certs are individually allocated
   // and that ownership is transferred to the caller upon return.
@@ -149,7 +147,10 @@ class NET_EXPORT DefaultServerBoundCertStore::PersistentStore
   virtual void Flush(const base::Closure& completion_task) = 0;
 
  protected:
+  friend class base::RefCountedThreadSafe<PersistentStore>;
+
   PersistentStore();
+  virtual ~PersistentStore();
 
  private:
   DISALLOW_COPY_AND_ASSIGN(PersistentStore);