RefCounted types should not have public destructors, delegate cleanup

chrome/browser/safe_browsing/safe_browsing_service.h

 // Copyright (c) 2012 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.
 //
 // The Safe Browsing service is responsible for downloading anti-phishing and
 // anti-malware tables and checking urls against them.
 
 #ifndef CHROME_BROWSER_SAFE_BROWSING_SAFE_BROWSING_SERVICE_H_
 #define CHROME_BROWSER_SAFE_BROWSING_SAFE_BROWSING_SERVICE_H_
 #pragma once
@@ skipping 121 matching lines @@
    protected:
     Observer() {}
     virtual ~Observer() {}
 
    private:
     DISALLOW_COPY_AND_ASSIGN(Observer);
   };
 
   class Client {
    public:
-    virtual ~Client() {}
-
     void OnSafeBrowsingResult(const SafeBrowsingCheck& check);
 
    protected:
+    virtual ~Client() {}
+
     // Called when the result of checking a browse URL is known.
     virtual void OnBrowseUrlCheckResult(const GURL& url,
                                         UrlCheckResult result) {}
 
     // Called when the result of checking a download URL is known.
     virtual void OnDownloadUrlCheckResult(const std::vector<GURL>& url_chain,
                                           UrlCheckResult result) {}
 
     // Called when the result of checking a download binary hash is known.
     virtual void OnDownloadHashCheckResult(const std::string& hash,
@@ skipping 435 matching lines @@
 class SafeBrowsingServiceFactory {
  public:
   SafeBrowsingServiceFactory() { }
   virtual ~SafeBrowsingServiceFactory() { }
   virtual SafeBrowsingService* CreateSafeBrowsingService() = 0;
  private:
   DISALLOW_COPY_AND_ASSIGN(SafeBrowsingServiceFactory);
 };
 
 #endif  // CHROME_BROWSER_SAFE_BROWSING_SAFE_BROWSING_SERVICE_H_