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

chrome/browser/history/top_sites.cc

 // 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.
 
 #include "chrome/browser/history/top_sites.h"
 
 #include <algorithm>
 #include <set>
 
 #include "base/bind.h"
@@ skipping 97 matching lines @@
       }
     }
     return true;
   }
 
   virtual void DoneRunOnMainThread() {
     top_sites_->FinishHistoryMigration(data_);
   }
 
  private:
+  virtual ~LoadThumbnailsFromHistoryTask() {}
+
   bool ShouldFetchThumbnailFor(const GURL& url) {
     return ignore_urls_.find(url.spec()) == ignore_urls_.end();
   }
 
   // Set of URLs we don't load thumbnails for. This is created on the UI thread
   // and used on the history thread.
   std::set<std::string> ignore_urls_;
 
   scoped_refptr<TopSites> top_sites_;
 
@@ skipping 765 matching lines @@
   SetTopSites(pages);
 
   // Used only in testing.
   content::NotificationService::current()->Notify(
       chrome::NOTIFICATION_TOP_SITES_UPDATED,
       content::Source<TopSites>(this),
       content::Details<CancelableRequestProvider::Handle>(&handle));
 }
 
 }  // namespace history