RefCounted types should not have public destructors, webkit/

webkit/quota/mock_quota_manager.cc

Index: webkit/quota/mock_quota_manager.cc
diff --git a/webkit/quota/mock_quota_manager.cc b/webkit/quota/mock_quota_manager.cc
index bac3fcc02a56b034b44c0f7d39707d1dc5a6fe54..595c43a56d60c3536f748ee131bfa500ce613a26 100644
--- a/webkit/quota/mock_quota_manager.cc
+++ b/webkit/quota/mock_quota_manager.cc
@@ -27,9 +27,13 @@ class MockQuotaManager::GetModifiedSinceTask : public QuotaThreadTask {
       : QuotaThreadTask(manager, manager->io_thread_),
         origins_(origins),
         type_(type),
-        callback_(callback) {}
+        callback_(callback) {
+  }
 
  protected:
+  virtual ~GetModifiedSinceTask() {}
+
+  // QuotaThreadTask:
   virtual void RunOnTargetThread() OVERRIDE {}
 
   virtual void Completed() OVERRIDE {
@@ -53,9 +57,13 @@ class MockQuotaManager::DeleteOriginDataTask : public QuotaThreadTask {
   DeleteOriginDataTask(MockQuotaManager* manager,
                        const StatusCallback& callback)
       : QuotaThreadTask(manager, manager->io_thread_),
-        callback_(callback) {}
+        callback_(callback) {
+  }
 
  protected:
+  virtual ~DeleteOriginDataTask() {}
+
+  // QuotaThreadTask:
   virtual void RunOnTargetThread() OVERRIDE {}
 
   virtual void Completed() OVERRIDE {
@@ -95,8 +103,6 @@ MockQuotaManager::MockQuotaManager(
         special_storage_policy) {
 }
 
-MockQuotaManager::~MockQuotaManager() {}
-
 bool MockQuotaManager::AddOrigin(
     const GURL& origin,
     StorageType type,
@@ -155,4 +161,6 @@ void MockQuotaManager::DeleteOriginData(
   make_scoped_refptr(new DeleteOriginDataTask(this, callback))->Start();
 }
 
+MockQuotaManager::~MockQuotaManager() {}
+
 }  // namespace quota