RefCounted types should not have public destructors, net/

net/base/default_server_bound_cert_store_unittest.cc

Index: net/base/default_server_bound_cert_store_unittest.cc
diff --git a/net/base/default_server_bound_cert_store_unittest.cc b/net/base/default_server_bound_cert_store_unittest.cc
index fb94234a0f08e09dc80347ae28c15b9030bd2c07..38053c9603a601f0a519655cfae29821b96d72b4 100644
--- a/net/base/default_server_bound_cert_store_unittest.cc
+++ b/net/base/default_server_bound_cert_store_unittest.cc
@@ -20,7 +20,6 @@ class MockPersistentStore
     : public DefaultServerBoundCertStore::PersistentStore {
  public:
   MockPersistentStore();
-  virtual ~MockPersistentStore();
 
   // DefaultServerBoundCertStore::PersistentStore implementation.
   virtual bool Load(
@@ -33,6 +32,9 @@ class MockPersistentStore
   virtual void SetClearLocalStateOnExit(bool clear_local_state) OVERRIDE;
   virtual void Flush(const base::Closure& completion_task) OVERRIDE;
 
+ protected:
+  virtual ~MockPersistentStore();
+
  private:
   typedef std::map<std::string, DefaultServerBoundCertStore::ServerBoundCert>
       ServerBoundCertMap;
@@ -42,8 +44,6 @@ class MockPersistentStore
 
 MockPersistentStore::MockPersistentStore() {}
 
-MockPersistentStore::~MockPersistentStore() {}
-
 bool MockPersistentStore::Load(
     std::vector<DefaultServerBoundCertStore::ServerBoundCert*>* certs) {
   ServerBoundCertMap::iterator it;
@@ -72,6 +72,8 @@ void MockPersistentStore::Flush(const base::Closure& completion_task) {
   NOTREACHED();
 }
 
+MockPersistentStore::~MockPersistentStore() {}
+
 TEST(DefaultServerBoundCertStoreTest, TestLoading) {
   scoped_refptr<MockPersistentStore> persistent_store(new MockPersistentStore);