Port HistoryService::ScheduleDBTask to CancelableTaskTracker

chrome/browser/sync/profile_sync_service_typed_url_unittest.cc

Index: chrome/browser/sync/profile_sync_service_typed_url_unittest.cc
diff --git a/chrome/browser/sync/profile_sync_service_typed_url_unittest.cc b/chrome/browser/sync/profile_sync_service_typed_url_unittest.cc
index c3780e5ab4644421901481a3c46d903fe1c848c3..d4a56885ed14a18588c94ff04b66287a14efc744 100644
--- a/chrome/browser/sync/profile_sync_service_typed_url_unittest.cc
+++ b/chrome/browser/sync/profile_sync_service_typed_url_unittest.cc
@@ -113,8 +113,8 @@ class HistoryServiceMock : public HistoryService {
  public:
   explicit HistoryServiceMock(history::HistoryClient* client, Profile* profile)
       : HistoryService(client, profile) {}
-  MOCK_METHOD2(ScheduleDBTask, void(history::HistoryDBTask*,
-                                    CancelableRequestConsumerBase*));
+  MOCK_METHOD2(ScheduleDBTask, void(scoped_refptr<history::HistoryDBTask>,
+                                    base::CancelableTaskTracker*));
   MOCK_METHOD0(Shutdown, void());
 
   void ShutdownBaseService() {
@@ -151,17 +151,17 @@ class TestTypedUrlModelAssociator : public TypedUrlModelAssociator {
 };
 
 void RunOnDBThreadCallback(HistoryBackend* backend,
-                           history::HistoryDBTask* task) {
+                           scoped_refptr<history::HistoryDBTask> task) {
   task->RunOnDBThread(backend, NULL);
 }
 
 ACTION_P2(RunTaskOnDBThread, thread, backend) {
   // ScheduleDBTask takes ownership of its task argument, so we

[blundell, 2014/06/26 09:55:39]

this comment can be killed now?

[sdefresne, 2014/06/28 08:14:26]

Done.

   // should, too.
-  scoped_refptr<history::HistoryDBTask> task(arg0);
-  thread->message_loop()->PostTask(
-      FROM_HERE, base::Bind(&RunOnDBThreadCallback, base::Unretained(backend),
-                            task));
+  thread->message_loop()->PostTask(FROM_HERE,
+                                   base::Bind(&RunOnDBThreadCallback,
+                                              base::Unretained(backend),
+                                              arg0));
 }
 
 ACTION_P2(ShutdownHistoryService, thread, service) {