Remove MessageLoop::current() from history_service.cc

components/history/core/browser/history_backend.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_HISTORY_CORE_BROWSER_HISTORY_BACKEND_H_
 #define COMPONENTS_HISTORY_CORE_BROWSER_HISTORY_BACKEND_H_
 
 #include <stddef.h>
 #include <stdint.h>
 
@@ skipping 23 matching lines @@
 #include "components/history/core/browser/visit_tracker.h"
 #include "sql/init_status.h"
 
 class HistoryURLProvider;
 struct HistoryURLProviderParams;
 class SkBitmap;
 class TestingProfile;
 struct ThumbnailScore;
 
 namespace base {
 class MessageLoop;

[sdefresne, 2016/10/13 17:25:35]

I think you can remove this line.

[fdoray, 2016/10/13 18:30:27]

Done.

 class SingleThreadTaskRunner;
 }
 
 namespace history {
 class CommitLaterTask;
 struct DownloadRow;
 class HistoryBackendClient;
 class HistoryBackendDBBaseTest;
 class HistoryBackendObserver;
 class HistoryBackendTest;
@@ skipping 395 matching lines @@
   // This object will TAKE OWNERSHIP of the given data pointer, and will
   // delete the object if it is changed or the object is destroyed.
   base::SupportsUserData::Data* GetUserData(const void* key) const;
   void SetUserData(const void* key, base::SupportsUserData::Data* data);
 
   // Testing -------------------------------------------------------------------
 
   // Sets the task to run and the message loop to run it on when this object
   // is destroyed. See HistoryService::SetOnBackendDestroyTask for a more
   // complete description.
-  void SetOnBackendDestroyTask(base::MessageLoop* message_loop,
-                               const base::Closure& task);
+  void SetOnBackendDestroyTask(
+      scoped_refptr<base::SingleThreadTaskRunner> task_runner,
+      const base::Closure& task);
 
   // Adds the given rows to the database if it doesn't exist. A visit will be
   // added for each given URL at the last visit time in the URLRow if the
   // passed visit type != SOURCE_SYNCED (the sync code manages visits itself).
   // Each visit will have the visit_source type set.
   void AddPagesWithDetails(const URLRows& info, VisitSource visit_source);
 
 #if defined(UNIT_TEST)
   HistoryDatabase* db() const { return db_.get(); }
 
@@ skipping 373 matching lines @@
   //
   // As with AddPage, the last item in the redirect chain will be the
   // destination of the redirect (i.e., the key into recent_redirects_);
   typedef base::MRUCache<GURL, RedirectList> RedirectCache;
   RedirectCache recent_redirects_;
 
   // Timestamp of the first entry in our database.
   base::Time first_recorded_time_;
 
   // When set, this is the task that should be invoked on destruction.
-  base::MessageLoop* backend_destroy_message_loop_;
+  scoped_refptr<base::SingleThreadTaskRunner> backend_destroy_task_runner_;
   base::Closure backend_destroy_task_;
 
   // Tracks page transition types.
   VisitTracker tracker_;
 
   // A boolean variable to track whether we have already purged obsolete segment
   // data.
   bool segment_queried_;
 
   // List of QueuedHistoryDBTasks to run;
@@ skipping 26 matching lines @@
 
   // List of observers
   base::ObserverList<HistoryBackendObserver> observers_;
 
   DISALLOW_COPY_AND_ASSIGN(HistoryBackend);
 };
 
 }  // namespace history
 
 #endif  // COMPONENTS_HISTORY_CORE_BROWSER_HISTORY_BACKEND_H_