Remove remaining calls to deprecated MessageLoop methods.

remoting/protocol/connection_tester.cc

 // 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.
 
 #include "remoting/protocol/connection_tester.h"
 
 #include "base/bind.h"
 #include "base/message_loop/message_loop.h"
+#include "base/single_thread_task_runner.h"
 #include "net/base/io_buffer.h"
 #include "net/base/net_errors.h"
 #include "remoting/proto/video.pb.h"
 #include "remoting/protocol/message_pipe.h"
 #include "remoting/protocol/message_serialization.h"
 #include "remoting/protocol/p2p_datagram_socket.h"
 #include "remoting/protocol/p2p_stream_socket.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 namespace remoting {
@@ skipping 30 matching lines @@
 
   output_buffer_->SetOffset(0);
   ASSERT_EQ(test_data_size_, output_buffer_->size());
 
   EXPECT_EQ(0, memcmp(output_buffer_->data(),
                       input_buffer_->StartOfBuffer(), test_data_size_));
 }
 
 void StreamConnectionTester::Done() {
   done_ = true;
-  message_loop_->PostTask(FROM_HERE, base::MessageLoop::QuitWhenIdleClosure());
+  message_loop_->task_runner()->PostTask(
+      FROM_HERE, base::MessageLoop::QuitWhenIdleClosure());
 }
 
 void StreamConnectionTester::InitBuffers() {
   output_buffer_ = new net::DrainableIOBuffer(
       new net::IOBuffer(test_data_size_), test_data_size_);
   for (int i = 0; i < test_data_size_; ++i) {
     output_buffer_->data()[i] = static_cast<char>(i);
   }
 
   input_buffer_ = new net::GrowableIOBuffer();
@@ skipping 60 matching lines @@
       Done();
   }
 }
 
 DatagramConnectionTester::DatagramConnectionTester(
     P2PDatagramSocket* client_socket,
     P2PDatagramSocket* host_socket,
     int message_size,
     int message_count,
     int delay_ms)
     : message_loop_(base::MessageLoop::current()),

[gab, 2016/06/30 20:28:18]

This could be TaskRunnerHandle and then no need for MessageLoop anywhere in this
file.

[fdoray, 2016/07/04 13:59:46]

Done.

       host_socket_(host_socket),
       client_socket_(client_socket),
       message_size_(message_size),
       message_count_(message_count),
       delay_ms_(delay_ms),
       done_(false),
       write_errors_(0),
       read_errors_(0),
       packets_sent_(0),
       packets_received_(0),
@@ skipping 16 matching lines @@
   EXPECT_EQ(0, bad_packets_received_);
 
   // Verify that we've received at least one packet.
   EXPECT_GT(packets_received_, 0);
   VLOG(0) << "Received " << packets_received_ << " packets out of "
           << message_count_;
 }
 
 void DatagramConnectionTester::Done() {
   done_ = true;
-  message_loop_->PostTask(FROM_HERE, base::MessageLoop::QuitWhenIdleClosure());
+  message_loop_->task_runner()->PostTask(
+      FROM_HERE, base::MessageLoop::QuitWhenIdleClosure());
 }
 
 void DatagramConnectionTester::DoWrite() {
   if (packets_sent_ >= message_count_) {
     Done();
     return;
   }
 
   scoped_refptr<net::IOBuffer> packet(new net::IOBuffer(message_size_));
   for (int i = 0; i < message_size_; ++i) {
@@ skipping 14 matching lines @@
 }
 
 void DatagramConnectionTester::HandleWriteResult(int result) {
   if (result <= 0 && result != net::ERR_IO_PENDING) {
     LOG(ERROR) << "Received error " << result << " when trying to write";
     write_errors_++;
     Done();
   } else if (result > 0) {
     EXPECT_EQ(message_size_, result);
     packets_sent_++;
-    message_loop_->PostDelayedTask(
+    message_loop_->task_runner()->PostDelayedTask(
         FROM_HERE,
         base::Bind(&DatagramConnectionTester::DoWrite, base::Unretained(this)),
         base::TimeDelta::FromMilliseconds(delay_ms_));
   }
 }
 
 void DatagramConnectionTester::DoRead() {
   int result = 1;
   while (result > 0) {
     int kReadSize = message_size_ * 2;
@@ skipping 73 matching lines @@
 void MessagePipeConnectionTester::OnMessageReceived(
     std::unique_ptr<CompoundBuffer> message) {
   received_messages_.push_back(ParseMessage<VideoPacket>(message.get()));
   if (received_messages_.size() >= sent_messages_.size()) {
     run_loop_.Quit();
   }
 }
 
 }  // namespace protocol
 }  // namespace remoting