RefCounted types should not have public destructors, chrome/browser/ part 1

chrome/browser/net/crl_set_fetcher.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.
 
 #ifndef CHROME_BROWSER_NET_CRL_SET_FETCHER_H_
 #define CHROME_BROWSER_NET_CRL_SET_FETCHER_H_
 #pragma once
 
 #include <string>
 
@@ skipping 11 matching lines @@
 namespace net {
 class CRLSet;
 }
 
 class ComponentUpdateService;
 
 class CRLSetFetcher : public ComponentInstaller,
                       public base::RefCountedThreadSafe<CRLSetFetcher> {
  public:
   CRLSetFetcher();
-  virtual ~CRLSetFetcher();
 
   void StartInitialLoad(ComponentUpdateService* cus);
 
   // ComponentInstaller interface
   virtual void OnUpdateError(int error) OVERRIDE;
   virtual bool Install(base::DictionaryValue* manifest,
                        const FilePath& unpack_path) OVERRIDE;
 
  private:
+  friend class base::RefCountedThreadSafe<CRLSetFetcher>;
+
+  virtual ~CRLSetFetcher();
+
   // GetCRLSetFilePath gets the path of the CRL set file in the user data
   // dir.
   bool GetCRLSetFilePath(FilePath* path) const;
 
   // DoInitialLoadFromDisk runs on the FILE thread and attempts to load a CRL
   // set from the user-data dir. It then registers this object as a component
   // in order to get updates.
   void DoInitialLoadFromDisk();
 
   // LoadFromDisk runs on the FILE thread and attempts to load a CRL set
@@ skipping 11 matching lines @@
   ComponentUpdateService* cus_;
 
   // We keep a pointer to the current CRLSet for use on the FILE thread when
   // applying delta updates.
   scoped_refptr<net::CRLSet> crl_set_;
 
   DISALLOW_COPY_AND_ASSIGN(CRLSetFetcher);
 };
 
 #endif  // CHROME_BROWSER_NET_CRL_SET_FETCHER_H_