[Offline pages] Remove references to MessageLoop from components/offline_pages

components/offline_pages/offline_page_metadata_store_impl.cc

 // Copyright 2015 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.
 
 #include "components/offline_pages/offline_page_metadata_store_impl.h"
 
 #include <string>
 #include <vector>
 
 #include "base/bind.h"
 #include "base/files/file_path.h"
 #include "base/location.h"
-#include "base/message_loop/message_loop.h"
 #include "base/metrics/histogram_macros.h"
 #include "base/sequenced_task_runner.h"
 #include "base/strings/string_number_conversions.h"
 #include "base/strings/utf_string_conversions.h"
 #include "base/thread_task_runner_handle.h"
 #include "components/leveldb_proto/proto_database_impl.h"
 #include "components/offline_pages/offline_page_item.h"
 #include "components/offline_pages/proto/offline_pages.pb.h"
 #include "third_party/leveldatabase/env_chromium.h"
 #include "third_party/leveldatabase/src/include/leveldb/db.h"
@@ skipping 185 matching lines @@
 void OfflinePageMetadataStoreImpl::UpdateEntries(
     scoped_ptr<ProtoDatabase<OfflinePageEntry>::KeyEntryVector> entries_to_save,
     scoped_ptr<std::vector<std::string>> keys_to_remove,
     const UpdateCallback& callback) {
   if (!database_.get()) {
     // Failing fast here, because DB is not initialized, and there is nothing
     // that can be done about it.
     // Callback is invoked through message loop to avoid improper retry and
     // simplify testing.
     DVLOG(1) << "Offline pages database not available in UpdateEntries.";
-    base::MessageLoop::current()->PostTask(FROM_HERE,
-                                           base::Bind(callback, false));
+    base::ThreadTaskRunnerHandle::Get()->PostTask(FROM_HERE,

[jianli, 2015/12/08 23:25:25]

nit: include base/thread_task_runner_handle.h

[fgorski, 2015/12/08 23:39:14]

It is actually already there, but it was not used :) I noticed that the first
time I was trying to add it.

+                                                  base::Bind(callback, false));
     return;
   }
 
   database_->UpdateEntries(
       entries_to_save.Pass(), keys_to_remove.Pass(),
       base::Bind(&OfflinePageMetadataStoreImpl::UpdateDone,
                  weak_ptr_factory_.GetWeakPtr(),
                  callback));
 }
 
@@ skipping 17 matching lines @@
 
 void OfflinePageMetadataStoreImpl::ResetDone(
     const ResetCallback& callback,
     bool success) {
   database_.reset();
   weak_ptr_factory_.InvalidateWeakPtrs();
   callback.Run(success);
 }
 
 }  // namespace offline_pages