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

chrome/browser/safe_browsing/malware_details_cache.h

 // 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.
 
 #ifndef CHROME_BROWSER_SAFE_BROWSING_MALWARE_DETAILS_CACHE_H_
 #define CHROME_BROWSER_SAFE_BROWSING_MALWARE_DETAILS_CACHE_H_
 #pragma once
 
 // A class that gets malware details from the HTTP Cache.
 // An instance of this class is generated by MalwareDetails.
@@ skipping 20 matching lines @@
   std::string,
   linked_ptr<safe_browsing::ClientMalwareReportRequest::Resource> > ResourceMap;
 }
 
 class MalwareDetailsCacheCollector
     : public base::RefCountedThreadSafe<MalwareDetailsCacheCollector>,
       public content::URLFetcherDelegate {
 
  public:
   MalwareDetailsCacheCollector();
-  virtual ~MalwareDetailsCacheCollector();
 
   // We use |request_context_getter|, we modify |resources| and
   // |result|, and we call |callback|, so they must all remain alive
   // for the lifetime of this object.
   void StartCacheCollection(
       net::URLRequestContextGetter* request_context_getter,
       safe_browsing::ResourceMap* resources,
       bool* result,
       const base::Closure& callback);
 
   // Returns whether or not StartCacheCollection has been called.
   bool HasStarted();
 
  protected:
   // Implementation of URLFetcher::Delegate. Called after the request
   // completes (either successfully or with failure).
   virtual void OnURLFetchComplete(const content::URLFetcher* source) OVERRIDE;
 
  private:
+  friend class base::RefCountedThreadSafe<MalwareDetailsCacheCollector>;
+
+  virtual ~MalwareDetailsCacheCollector();
+
   // Points to the url for which we are fetching the HTTP cache entry or
   // redirect chain.
   safe_browsing::ResourceMap::iterator resources_it_;
 
   // Points to the resources_ map in the MalwareDetails.
   safe_browsing::ResourceMap* resources_;
 
   // Points to the cache_result_ in the MalwareDetails.
   bool* result_;
 
@@ skipping 28 matching lines @@
       const std::string& data);
 
   // Called when we are done.
   void AllDone(bool success);
 
   // Advances to the next entry in resources_it_.
   void AdvanceEntry();
 };
 
 #endif  // CHROME_BROWSER_SAFE_BROWSING_MALWARE_DETAILS_CACHE_H_