Change HistoryService::ScheduleAutocomplete interface

chrome/browser/history/history_backend.cc

Index: chrome/browser/history/history_backend.cc
diff --git a/chrome/browser/history/history_backend.cc b/chrome/browser/history/history_backend.cc
index c30c6aa80b20e5a17b8b6b31510a802321a17022..fdcf3e425cee9d63e574d4bfa6e09e3a58ed7263 100644
--- a/chrome/browser/history/history_backend.cc
+++ b/chrome/browser/history/history_backend.cc
@@ -23,7 +23,6 @@
 #include "base/strings/string_util.h"
 #include "base/strings/utf_string_conversions.h"
 #include "base/time/time.h"
-#include "chrome/browser/autocomplete/history_url_provider.h"
 #include "chrome/browser/chrome_notification_types.h"
 #include "chrome/browser/favicon/favicon_changed_details.h"
 #include "chrome/browser/history/download_row.h"
@@ -1540,10 +1539,10 @@ bool HistoryBackend::GetMostRecentRedirectsTo(
   return true;
 }
 
-void HistoryBackend::ScheduleAutocomplete(HistoryURLProvider* provider,
-                                          HistoryURLProviderParams* params) {
+void HistoryBackend::ScheduleAutocomplete(const base::Callback<
+    void(history::HistoryBackend*, history::URLDatabase*)>& callback) {
   // ExecuteWithDB should handle the NULL database case.

[blundell, 2014/07/08 09:10:05]

This comment is out of date now; instead, the comment on the declaration should
specify that the callback must be able to handle the case where the database is
null.

[nshaik, 2014/07/08 18:53:49]

Done.

-  provider->ExecuteWithDB(this, db_.get(), params);
+  callback.Run(this, db_.get());
 }
 
 void HistoryBackend::DeleteFTSIndexDatabases() {