RefCounted types should not have public destructors, webkit/

webkit/appcache/appcache_storage_impl_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 <stack>
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/callback.h"
 #include "base/message_loop.h"
@@ skipping 139 matching lines @@
         return;
       }
       CallCallback(callback);
     }
 
     void CallCallback(const GetUsageAndQuotaCallback& callback) {
       callback.Run(quota::kQuotaStatusOk, 0, kMockQuota);
     }
 
     bool async_;
+
+   protected:
+    virtual ~MockQuotaManager() {}
   };
 
   class MockQuotaManagerProxy : public quota::QuotaManagerProxy {
    public:
     MockQuotaManagerProxy()
         : QuotaManagerProxy(NULL, NULL),
           notify_storage_accessed_count_(0),
           notify_storage_modified_count_(0),
           last_delta_(0),
           mock_manager_(new MockQuotaManager) {
@@ skipping 23 matching lines @@
     // Not needed for our tests.
     virtual void RegisterClient(quota::QuotaClient* client) OVERRIDE {}
     virtual void NotifyOriginInUse(const GURL& origin) OVERRIDE {}
     virtual void NotifyOriginNoLongerInUse(const GURL& origin) OVERRIDE {}
 
     int notify_storage_accessed_count_;
     int notify_storage_modified_count_;
     GURL last_origin_;
     int last_delta_;
     scoped_refptr<MockQuotaManager> mock_manager_;
+
+   protected:
+    virtual ~MockQuotaManagerProxy() {}
   };
 
   template <class Method>
   void RunMethod(Method method) {
     (this->*method)();
   }
 
   // Helper callback to run a test on our io_thread. The io_thread is spun up
   // once and reused for all tests.
   template <class Method>
@@ skipping 1190 matching lines @@
 }
 
 TEST_F(AppCacheStorageImplTest, FindMainResponseExclusionsInWorkingSet) {
   RunTestOnIOThread(
       &AppCacheStorageImplTest::FindMainResponseExclusionsInWorkingSet);
 }
 
 // That's all folks!
 
 }  // namespace appcache