base: Remove most uses of MessageLoopProxy

base/posix/unix_domain_socket_linux_unittest.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 <sys/socket.h>
 #include <sys/types.h>
 #include <unistd.h>
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/files/file_util.h"
 #include "base/files/scoped_file.h"
+#include "base/location.h"
 #include "base/memory/scoped_vector.h"
 #include "base/pickle.h"
 #include "base/posix/unix_domain_socket_linux.h"
+#include "base/single_thread_task_runner.h"
 #include "base/synchronization/waitable_event.h"
 #include "base/threading/thread.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 namespace base {
 
 namespace {
 
 TEST(UnixDomainSocketTest, SendRecvMsgAbortOnReplyFDClose) {
   Thread message_thread("UnixDomainSocketTest");
   ASSERT_TRUE(message_thread.Start());
 
   int fds[2];
   ASSERT_EQ(0, socketpair(AF_UNIX, SOCK_SEQPACKET, 0, fds));
   ScopedFD scoped_fd0(fds[0]);
   ScopedFD scoped_fd1(fds[1]);
 
   // Have the thread send a synchronous message via the socket.
   Pickle request;
-  message_thread.message_loop()->PostTask(
+  message_thread.message_loop()->task_runner()->PostTask(

[danakj, 2015/04/21 20:16:30]

thread.task_runner() here too? I think there's some more too, can your script
find these?

[Sami, 2015/04/23 17:48:24]

Yep, these should be fixed now.

       FROM_HERE,
-      Bind(IgnoreResult(&UnixDomainSocket::SendRecvMsg),
-           fds[1], static_cast<uint8_t*>(NULL), 0U, static_cast<int*>(NULL),
-           request));
+      Bind(IgnoreResult(&UnixDomainSocket::SendRecvMsg), fds[1],
+           static_cast<uint8_t*>(NULL), 0U, static_cast<int*>(NULL), request));
 
   // Receive the message.
   ScopedVector<base::ScopedFD> message_fds;
   uint8_t buffer[16];
   ASSERT_EQ(static_cast<int>(request.size()),
             UnixDomainSocket::RecvMsg(fds[0], buffer, sizeof(buffer),
                                       &message_fds));
   ASSERT_EQ(1U, message_fds.size());
 
   // Close the reply FD.
   message_fds.clear();
 
   // Check that the thread didn't get blocked.
   WaitableEvent event(false, false);
-  message_thread.message_loop()->PostTask(
-      FROM_HERE,
-      Bind(&WaitableEvent::Signal, Unretained(&event)));
+  message_thread.message_loop()->task_runner()->PostTask(
+      FROM_HERE, Bind(&WaitableEvent::Signal, Unretained(&event)));
   ASSERT_TRUE(event.TimedWait(TimeDelta::FromMilliseconds(5000)));
 }
 
 TEST(UnixDomainSocketTest, SendRecvMsgAvoidsSIGPIPE) {
   // Make sure SIGPIPE isn't being ignored.
   struct sigaction act = {}, oldact;
   act.sa_handler = SIG_DFL;
   ASSERT_EQ(0, sigaction(SIGPIPE, &act, &oldact));
   int fds[2];
   ASSERT_EQ(0, socketpair(AF_UNIX, SOCK_SEQPACKET, 0, fds));
@@ skipping 85 matching lines @@
   const ssize_t nread = UnixDomainSocket::RecvMsgWithPid(
       recv_sock.get(), &ch, sizeof(ch), &recv_fds, &sender_pid);
   ASSERT_EQ(0, nread);
   ASSERT_EQ(-1, sender_pid);
   ASSERT_EQ(0U, recv_fds.size());
 }
 
 }  // namespace
 
 }  // namespace base