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

chrome/browser/safe_browsing/malware_details_history.cc

Index: chrome/browser/safe_browsing/malware_details_history.cc
diff --git a/chrome/browser/safe_browsing/malware_details_history.cc b/chrome/browser/safe_browsing/malware_details_history.cc
index 3947f36fd8332e3a72d27bdb55ffcd8be973500f..1cf11ef3d94696953c0b80ef832393e4f27e2cad 100644
--- a/chrome/browser/safe_browsing/malware_details_history.cc
+++ b/chrome/browser/safe_browsing/malware_details_history.cc
@@ -30,9 +30,6 @@ MalwareDetailsRedirectsCollector::MalwareDetailsRedirectsCollector(
   }
 }
 
-MalwareDetailsRedirectsCollector::~MalwareDetailsRedirectsCollector() {
-}
-
 void MalwareDetailsRedirectsCollector::StartHistoryCollection(
     const std::vector<GURL>& urls,
     const base::Closure& callback) {
@@ -51,6 +48,27 @@ void MalwareDetailsRedirectsCollector::StartHistoryCollection(
                  this, urls));
 }
 
+bool MalwareDetailsRedirectsCollector::HasStarted() const {
+  return has_started_;
+}
+
+const std::vector<safe_browsing::RedirectChain>&
+MalwareDetailsRedirectsCollector::GetCollectedUrls() const {
+  return redirects_urls_;
+}
+
+void MalwareDetailsRedirectsCollector::Observe(
+    int type,
+    const content::NotificationSource& source,
+    const content::NotificationDetails& details) {
+  DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
+  DCHECK_EQ(type, chrome::NOTIFICATION_PROFILE_DESTROYED);
+  DVLOG(1) << "Profile gone.";
+  profile_ = NULL;
+}
+
+MalwareDetailsRedirectsCollector::~MalwareDetailsRedirectsCollector() {}
+
 void MalwareDetailsRedirectsCollector::StartGetRedirects(
         const std::vector<GURL>& urls) {
   // History access from profile needs to happen in UI thread
@@ -109,27 +127,8 @@ void MalwareDetailsRedirectsCollector::OnGotQueryRedirectsTo(
   GetRedirects(*urls_it_);
 }
 
-bool MalwareDetailsRedirectsCollector::HasStarted() const {
-  return has_started_;
-}
-
 void MalwareDetailsRedirectsCollector::AllDone() {
   DVLOG(1) << "AllDone";
   BrowserThread::PostTask(BrowserThread::IO, FROM_HERE, callback_);
   callback_.Reset();
 }
-
-const std::vector<safe_browsing::RedirectChain>&
-MalwareDetailsRedirectsCollector::GetCollectedUrls() const {
-  return redirects_urls_;
-}
-
-void MalwareDetailsRedirectsCollector::Observe(
-    int type,
-    const content::NotificationSource& source,
-    const content::NotificationDetails& details) {
-  DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
-  DCHECK_EQ(type, chrome::NOTIFICATION_PROFILE_DESTROYED);
-  DVLOG(1) << "Profile gone.";
-  profile_ = NULL;
-}