Remove MessageLoop::current() from history_url_provider.cc

components/omnibox/browser/history_url_provider.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef COMPONENTS_OMNIBOX_BROWSER_HISTORY_URL_PROVIDER_H_
 #define COMPONENTS_OMNIBOX_BROWSER_HISTORY_URL_PROVIDER_H_
 
 #include <stddef.h>
 
 #include <string>
 #include <vector>
 
 #include "base/compiler_specific.h"
 #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"
 #include "components/omnibox/browser/history_provider.h"
 #include "components/omnibox/browser/omnibox_field_trial.h"
 #include "components/search_engines/template_url.h"
 
 class AutocompleteProviderListener;
 class SearchTermsData;
 
@@ skipping 73 matching lines @@
     NEITHER,
   };
 
   HistoryURLProviderParams(const AutocompleteInput& input,
                            bool trim_http,
                            const AutocompleteMatch& what_you_typed_match,
                            TemplateURL* default_search_provider,
                            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.
   AutocompleteInput input;
 
   // Should inline autocompletion be disabled? This is initalized from
   // |input.prevent_inline_autocomplete()|, but set to false is the input
   // contains trailing white space.
   bool prevent_inline_autocomplete;
 
@@ skipping 221 matching lines @@
 
   // Params controlling experimental behavior of this provider.
   HUPScoringParams scoring_params_;
 
   base::ThreadChecker thread_checker_;
 
   DISALLOW_COPY_AND_ASSIGN(HistoryURLProvider);
 };
 
 #endif  // COMPONENTS_OMNIBOX_BROWSER_HISTORY_URL_PROVIDER_H_