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

chrome/browser/safe_browsing/malware_details_history.h

Index: chrome/browser/safe_browsing/malware_details_history.h
diff --git a/chrome/browser/safe_browsing/malware_details_history.h b/chrome/browser/safe_browsing/malware_details_history.h
index b956aedaf764aa430bffa9528864b9d2142aae81..50efb772d2264827b062c93f6fc4dcbf848dd063 100644
--- a/chrome/browser/safe_browsing/malware_details_history.h
+++ b/chrome/browser/safe_browsing/malware_details_history.h
@@ -33,7 +33,6 @@ class MalwareDetailsRedirectsCollector
       public content::NotificationObserver {
  public:
   explicit MalwareDetailsRedirectsCollector(Profile* profile);
-  virtual ~MalwareDetailsRedirectsCollector();
 
   // Collects urls' redirects chain information from the history service.
   // We get access to history service via tab_contents in UI thread.
@@ -47,11 +46,29 @@ class MalwareDetailsRedirectsCollector
   // Returns the redirect urls we get from history service
   const std::vector<safe_browsing::RedirectChain>& GetCollectedUrls() const;
 
+  // content::NotificationObserver
+  virtual void Observe(int type,
+                       const content::NotificationSource& source,
+                       const content::NotificationDetails& details) OVERRIDE;
+
  private:
   friend struct content::BrowserThread::DeleteOnThread<
       content::BrowserThread::UI>;
   friend class base::DeleteHelper<MalwareDetailsRedirectsCollector>;
 
+  virtual ~MalwareDetailsRedirectsCollector();
+
+  void StartGetRedirects(const std::vector<GURL>& urls);
+  void GetRedirects(const GURL& url);
+  void OnGotQueryRedirectsTo(HistoryService::Handle handle,
+                             GURL url,
+                             bool success,
+                             history::RedirectList* redirect_list);
+
+  // Posts the callback method back to IO thread when redirects collecting
+  // is all done.
+  void AllDone();
+
   Profile* profile_;
   CancelableRequestConsumer request_consumer_;
 
@@ -71,21 +88,6 @@ class MalwareDetailsRedirectsCollector
 
   content::NotificationRegistrar registrar_;
 
-  void StartGetRedirects(const std::vector<GURL>& urls);
-  void GetRedirects(const GURL& url);
-  void OnGotQueryRedirectsTo(HistoryService::Handle handle,
-                             GURL url,
-                             bool success,
-                             history::RedirectList* redirect_list);
-
-  // Posts the callback method back to IO thread when redirects collecting
-  // is all done.
-  void AllDone();
-
-  virtual void Observe(int type,
-                       const content::NotificationSource& source,
-                       const content::NotificationDetails& details) OVERRIDE;
-
   DISALLOW_COPY_AND_ASSIGN(MalwareDetailsRedirectsCollector);
 };