Remove MessageLoop::current() from history_url_provider.cc

components/omnibox/browser/history_url_provider.h

Index: components/omnibox/browser/history_url_provider.h
diff --git a/components/omnibox/browser/history_url_provider.h b/components/omnibox/browser/history_url_provider.h
index 23ae4c4d6efaa92ef468cbb91c2bbd27b5e43c77..5d3b72205f266c8e9499005a242774569671afe7 100644
--- a/components/omnibox/browser/history_url_provider.h
+++ b/components/omnibox/browser/history_url_provider.h
@@ -14,6 +14,7 @@
 #include "base/gtest_prod_util.h"
 #include "base/macros.h"
 #include "base/synchronization/cancellation_flag.h"
+#include "base/threading/sequenced_task_runner_handle.h"
 #include "base/threading/thread_checker.h"
 #include "components/history/core/browser/history_match.h"
 #include "components/omnibox/browser/autocomplete_input.h"
@@ -107,7 +108,8 @@ struct HistoryURLProviderParams {
                            const SearchTermsData& search_terms_data);
   ~HistoryURLProviderParams();
 
-  base::MessageLoop* message_loop;
+  const scoped_refptr<base::SequencedTaskRunner> origin_task_runner =
+      base::SequencedTaskRunnerHandle::Get();

[Peter Kasting, 2016/10/19 18:07:11]

Nit: Can we initialize this in the constructor instead for consistency, as
that's where we init these other members?

(This might also allow forward-declaring SequencedTaskRunner instead of
#including a header in this header.)

[fdoray, 2016/10/20 13:50:40]

Done.

 
   // A copy of the autocomplete input. We need the copy since this object will
   // live beyond the original query while it runs on the history thread.