[chrome/browser/policy] Replace MessageLoopProxy usage with ThreadTaskRunnerHandle

chrome/browser/policy/cloud/test_request_interceptor.cc

 // Copyright (c) 2013 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 "chrome/browser/policy/cloud/test_request_interceptor.h"
 
 #include <limits>
 #include <queue>
 #include <vector>
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/message_loop/message_loop_proxy.h"
 #include "base/run_loop.h"
 #include "base/sequenced_task_runner.h"
+#include "base/thread_task_runner_handle.h"
 #include "content/public/browser/browser_thread.h"
 #include "net/base/net_errors.h"
 #include "net/base/upload_bytes_element_reader.h"
 #include "net/base/upload_data_stream.h"
 #include "net/base/upload_element_reader.h"
 #include "net/test/url_request/url_request_mock_http_job.h"
 #include "net/url_request/url_request_error_job.h"
 #include "net/url_request/url_request_filter.h"
 #include "net/url_request/url_request_interceptor.h"
 #include "net/url_request/url_request_test_job.h"
@@ skipping 252 matching lines @@
   size_t pending_size = std::numeric_limits<size_t>::max();
   PostToIOAndWait(base::Bind(&Delegate::GetPendingSize,
                              base::Unretained(delegate_),
                              &pending_size));
   return pending_size;
 }
 
 void TestRequestInterceptor::AddRequestServicedCallback(
     const base::Closure& callback) {
   base::Closure post_callback =
       base::Bind(base::IgnoreResult(&base::MessageLoopProxy::PostTask),

[pneubeck (no reviews), 2015/04/30 08:00:08]

this reads very inconsistent.
Shouldn't this be changed to
 ...IgnoreResult(&base::TaskRunner::PostTask)...
?

[Pranay, 2015/05/04 03:26:59]

Done.

-                 base::MessageLoopProxy::current(),
+                 base::ThreadTaskRunnerHandle::Get(),
                  FROM_HERE,
                  callback);
   PostToIOAndWait(base::Bind(&Delegate::AddRequestServicedCallback,
                              base::Unretained(delegate_),
                              post_callback));
 }
 
 void TestRequestInterceptor::PushJobCallback(const JobCallback& callback) {
   PostToIOAndWait(base::Bind(&Delegate::PushJobCallback,
                              base::Unretained(delegate_),
@@ skipping 23 matching lines @@
     const base::FilePath& file_path) {
   return base::Bind(&FileJobCallback, file_path);
 }
 
 void TestRequestInterceptor::PostToIOAndWait(const base::Closure& task) {
   io_task_runner_->PostTask(FROM_HERE, task);
   base::RunLoop run_loop;
   io_task_runner_->PostTask(
       FROM_HERE,
       base::Bind(
           base::IgnoreResult(&base::MessageLoopProxy::PostTask),

[pneubeck (no reviews), 2015/04/30 08:00:08]

ditto.

now you should be able to remove the include
"base/message_loop/message_loop_proxy.h"
as well.

[Pranay, 2015/05/04 03:26:59]

Done.

-          base::MessageLoopProxy::current(),
+          base::ThreadTaskRunnerHandle::Get(),
           FROM_HERE,
           run_loop.QuitClosure()));
   run_loop.Run();
 }
 
 }  // namespace policy