Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(3667)

Unified Diff: chrome/browser/history/top_sites_impl.cc

Issue 369503007: Remove dead code (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix windows compilation Created 6 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/history/top_sites_impl.h ('k') | chrome/browser/jumplist_win.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/history/top_sites_impl.cc
diff --git a/chrome/browser/history/top_sites_impl.cc b/chrome/browser/history/top_sites_impl.cc
index d1fd053df57c59a336fe0995a1d102431ddc496e..e3298d8f396d39dce803c4b6205ba1af286d6993 100644
--- a/chrome/browser/history/top_sites_impl.cc
+++ b/chrome/browser/history/top_sites_impl.cc
@@ -378,7 +378,7 @@ void TopSitesImpl::Shutdown() {
// Cancel all requests so that the service doesn't callback to us after we've
// invoked Shutdown (this could happen if we have a pending request and
// Shutdown is invoked).
- history_consumer_.CancelAllRequests();
+ cancelable_task_tracker_.TryCancelAll();
backend_->Shutdown();
}
@@ -444,10 +444,10 @@ void TopSitesImpl::DiffMostVisited(const MostVisitedURLList& old_list,
}
}
-CancelableRequestProvider::Handle TopSitesImpl::StartQueryForMostVisited() {
+base::CancelableTaskTracker::TaskId TopSitesImpl::StartQueryForMostVisited() {
DCHECK(loaded_);
if (!profile_)
- return 0;
+ return base::CancelableTaskTracker::kBadTaskId;
HistoryService* hs = HistoryServiceFactory::GetForProfile(
profile_, Profile::EXPLICIT_ACCESS);
@@ -460,7 +460,7 @@ CancelableRequestProvider::Handle TopSitesImpl::StartQueryForMostVisited() {
base::Unretained(this)),
&cancelable_task_tracker_);
}
- return 0;
+ return base::CancelableTaskTracker::kBadTaskId;
}
bool TopSitesImpl::IsKnownURL(const GURL& url) {
« no previous file with comments | « chrome/browser/history/top_sites_impl.h ('k') | chrome/browser/jumplist_win.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698