Port History::QueryMostVisitedURLs to use CancelableTaskTracker

chrome/browser/history/history_service.h

Index: chrome/browser/history/history_service.h
diff --git a/chrome/browser/history/history_service.h b/chrome/browser/history/history_service.h
index af915f93f5fbf17e389e412952fbfc2c4fcea9ef..1d03214e166fdf09ba0ed660ee7268b110904ebe 100644
--- a/chrome/browser/history/history_service.h
+++ b/chrome/browser/history/history_service.h
@@ -311,12 +311,14 @@ class HistoryService : public CancelableRequestProvider,
   // Request the |result_count| most visited URLs and the chain of
   // redirects leading to each of these URLs. |days_back| is the
   // number of days of history to use. Used by TopSites.
-  typedef base::Callback<void(Handle, history::MostVisitedURLList)>
+  typedef base::Callback<void(const history::MostVisitedURLList*)>

[blundell, 2014/06/26 09:21:54]

I think that we avoid const pointers in Chromium, although I don't know the
reasoning as to why.

       QueryMostVisitedURLsCallback;
 
-  Handle QueryMostVisitedURLs(int result_count, int days_back,
-                              CancelableRequestConsumerBase* consumer,
-                              const QueryMostVisitedURLsCallback& callback);
+  base::CancelableTaskTracker::TaskId QueryMostVisitedURLs(
+      int result_count,
+      int days_back,
+      const QueryMostVisitedURLsCallback& callback,
+      base::CancelableTaskTracker* tracker);
 
   // Request the |result_count| URLs filtered and sorted based on the |filter|.
   // If |extended_info| is true, additional data will be provided in the