Remove calls to deprecated MessageLoop methods in components.

components/nacl/loader/nacl_ipc_adapter_unittest.cc

 // Copyright 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 "components/nacl/loader/nacl_ipc_adapter.h"
 
 #include <stddef.h>
 #include <stdint.h>
 #include <string.h>
 
 #include <memory>
 
+#include "base/run_loop.h"
 #include "base/threading/platform_thread.h"
 #include "base/threading/simple_thread.h"
 #include "base/threading/thread_task_runner_handle.h"
 #include "ipc/ipc_test_sink.h"
 #include "native_client/src/public/nacl_desc_custom.h"
 #include "native_client/src/trusted/service_runtime/include/sys/fcntl.h"
 #include "ppapi/c/ppb_file_io.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 namespace {
@@ skipping 11 matching lines @@
     // not need real IPC for the tests.
     adapter_ = new NaClIPCAdapter(std::unique_ptr<IPC::Channel>(sink_),
                                   base::ThreadTaskRunnerHandle::Get().get());
   }
   void TearDown() override {
     sink_ = NULL;  // This pointer is actually owned by the IPCAdapter.
     adapter_ = NULL;
     // The adapter destructor has to post a task to destroy the Channel on the
     // IO thread. For the purposes of the test, we just need to make sure that
     // task gets run, or it will appear as a leak.
-    message_loop_.RunUntilIdle();
+    base::RunLoop().RunUntilIdle();
   }
 
  protected:
   int BlockingReceive(void* buf, size_t buf_size) {
     NaClImcMsgIoVec iov = {buf, buf_size};
     NaClImcTypedMsgHdr msg = {&iov, 1};
     return adapter_->BlockingReceive(&msg);
   }
 
   int Send(void* buf, size_t buf_size) {
@@ skipping 70 matching lines @@
   header->flags = 0;
   header->num_fds = 0;
   *reinterpret_cast<int*>(
       &buf[sizeof(NaClIPCAdapter::NaClMessageHeader)]) = value;
 
   int result = Send(buf, buf_size);
   EXPECT_EQ(buf_size, result);
 
   // Check that the message came out the other end in the test sink
   // (messages are posted, so we have to pump).
-  message_loop_.RunUntilIdle();
+  base::RunLoop().RunUntilIdle();
   ASSERT_EQ(1u, sink_->message_count());
   const IPC::Message* msg = sink_->GetMessageAt(0);
 
   EXPECT_EQ(sizeof(int), msg->payload_size());
   EXPECT_EQ(header->routing, msg->routing_id());
   EXPECT_EQ(header->type, msg->type());
 
   // Now test the partial send case. We should be able to break the message
   // into two parts and it should still work.
   sink_->ClearMessages();
   int first_chunk_size = 7;
   result = Send(buf, first_chunk_size);
   EXPECT_EQ(first_chunk_size, result);
 
   // First partial send should not have made any messages.
-  message_loop_.RunUntilIdle();
+  base::RunLoop().RunUntilIdle();
   ASSERT_EQ(0u, sink_->message_count());
 
   // Second partial send should do the same.
   int second_chunk_size = 2;
   result = Send(&buf[first_chunk_size], second_chunk_size);
   EXPECT_EQ(second_chunk_size, result);
-  message_loop_.RunUntilIdle();
+  base::RunLoop().RunUntilIdle();
   ASSERT_EQ(0u, sink_->message_count());
 
   // Send the rest of the message in a third chunk.
   int third_chunk_size = buf_size - first_chunk_size - second_chunk_size;
   result = Send(&buf[first_chunk_size + second_chunk_size],
                           third_chunk_size);
   EXPECT_EQ(third_chunk_size, result);
 
   // Last send should have generated one message.
-  message_loop_.RunUntilIdle();
+  base::RunLoop().RunUntilIdle();
   ASSERT_EQ(1u, sink_->message_count());
   msg = sink_->GetMessageAt(0);
   EXPECT_EQ(sizeof(int), msg->payload_size());
   EXPECT_EQ(header->routing, msg->routing_id());
   EXPECT_EQ(header->type, msg->type());
 }
 
 // Tests when a buffer is too small to receive the entire message.
 TEST_F(NaClIPCAdapterTest, PartialReceive) {
   int routing_id_1 = 0x89898989;
@@ skipping 62 matching lines @@
   header->routing = routing_id;
   header->type = type;
   header->flags = 0;
   header->num_fds = 0;
   *reinterpret_cast<int*>(
       &buf[sizeof(NaClIPCAdapter::NaClMessageHeader)]) = value;
 
   // Send too much data and make sure that the send fails.
   int result = Send(buf, big_buf_size);
   EXPECT_EQ(-1, result);
-  message_loop_.RunUntilIdle();
+  base::RunLoop().RunUntilIdle();
   ASSERT_EQ(0u, sink_->message_count());
 
   // Send too much data in two chunks and make sure that the send fails.
   int first_chunk_size = 7;
   result = Send(buf, first_chunk_size);
   EXPECT_EQ(first_chunk_size, result);
 
   // First partial send should not have made any messages.
-  message_loop_.RunUntilIdle();
+  base::RunLoop().RunUntilIdle();
   ASSERT_EQ(0u, sink_->message_count());
 
   int second_chunk_size = big_buf_size - first_chunk_size;
   result = Send(&buf[first_chunk_size], second_chunk_size);
   EXPECT_EQ(-1, result);
-  message_loop_.RunUntilIdle();
+  base::RunLoop().RunUntilIdle();
   ASSERT_EQ(0u, sink_->message_count());
 }
 
 // Tests that when the IPC channel reports an error, that waiting reads are
 // unblocked and return a -1 error code.
 TEST_F(NaClIPCAdapterTest, ReadWithChannelError) {
   // Have a background thread that waits a bit and calls the channel error
   // handler. This should wake up any waiting threads and immediately return
   // -1. There is an inherent race condition in that we can't be sure if the
   // other thread is actually waiting when this happens. This is OK, since the
@@ skipping 73 matching lines @@
   EXPECT_EQ(NACL_ABI_O_RDONLY,
       TranslatePepperFileReadWriteOpenFlagsForTesting(
           PP_FILEOPENFLAG_CREATE));
   EXPECT_EQ(NACL_ABI_O_RDONLY,
       TranslatePepperFileReadWriteOpenFlagsForTesting(
           PP_FILEOPENFLAG_TRUNCATE));
   EXPECT_EQ(NACL_ABI_O_RDONLY,
       TranslatePepperFileReadWriteOpenFlagsForTesting(
           PP_FILEOPENFLAG_EXCLUSIVE));
 }