Don't use base::MessageLoop::{Quit,QuitClosure} in ash/, chromeos/, device/

chromeos/process_proxy/process_proxy_unittest.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 <gtest/gtest.h>
 
 #include <string>
 
 #include "base/bind.h"
 #include "base/location.h"
@@ skipping 61 matching lines @@
     for (size_t i = 0; i < output.length(); i++) {
       // The character output[i] should be next in at least one of the lines we
       // are testing.
       valid = (ProcessReceivedCharacter(output[i], 0) ||
                ProcessReceivedCharacter(output[i], 1));
       EXPECT_TRUE(valid) << "Received: " << output;
     }
 
     if (!valid || TestSucceeded()) {
       base::ThreadTaskRunnerHandle::Get()->PostTask(
-          FROM_HERE, base::MessageLoop::QuitClosure());
+          FROM_HERE, base::MessageLoop::QuitWhenIdleClosure());
     }
   }
 
   void StartRegistryTest(ProcessProxyRegistry* registry) override {
     for (int i = 0; i < kTestLineNum; i++) {
       EXPECT_TRUE(registry->SendInput(pid_, kTestLineToSend));
     }
   }
 
  private:
@@ skipping 42 matching lines @@
       output_received_ = true;
       EXPECT_EQ(type, "stdout");
       EXPECT_EQ(output, "p");
       base::Process process =
           base::Process::DeprecatedGetProcessFromHandle(pid_);
       process.Terminate(0, true);
       return;
     }
     EXPECT_EQ("exit", type);
     base::ThreadTaskRunnerHandle::Get()->PostTask(
-        FROM_HERE, base::MessageLoop::QuitClosure());
+        FROM_HERE, base::MessageLoop::QuitWhenIdleClosure());
   }
 
   void StartRegistryTest(ProcessProxyRegistry* registry) override {
     EXPECT_TRUE(registry->SendInput(pid_, "p"));
   }
 
  private:
   bool output_received_;
 };
 
 class SigIntTestRunner : public TestRunner {
  public:
   ~SigIntTestRunner() override {}
 
   void SetupExpectations(pid_t pid) override { pid_ = pid; }
 
   void OnSomeRead(pid_t pid,
                   const std::string& type,
                   const std::string& output) override {
     EXPECT_EQ(pid_, pid);
     // We may receive ^C on stdout, but we don't care about that, as long as we
     // eventually received exit event.
     if (type == "exit") {
       base::ThreadTaskRunnerHandle::Get()->PostTask(
-          FROM_HERE, base::MessageLoop::QuitClosure());
+          FROM_HERE, base::MessageLoop::QuitWhenIdleClosure());
     }
   }
 
   void StartRegistryTest(ProcessProxyRegistry* registry) override {
     // Send SingInt and verify the process exited.
     EXPECT_TRUE(registry->SendInput(pid_, "\003"));
   }
 };
 
 }  // namespace
@@ skipping 23 matching lines @@
     EXPECT_NE(base::TERMINATION_STATUS_STILL_RUNNING, status);
     if (status == base::TERMINATION_STATUS_STILL_RUNNING) {
       base::Process process =
           base::Process::DeprecatedGetProcessFromHandle(pid_);
       process.Terminate(0, true);
     }
 
     registry_->ShutDown();
 
     base::ThreadTaskRunnerHandle::Get()->PostTask(
-        FROM_HERE, base::MessageLoop::QuitClosure());
+        FROM_HERE, base::MessageLoop::QuitWhenIdleClosure());
   }
 
   void RunTest() {
     base::ThreadTaskRunnerHandle::Get()->PostTask(
         FROM_HERE, base::Bind(&ProcessProxyTest::InitRegistryTest,
                               base::Unretained(this)));
 
     // Wait until all data from output watcher is received (QuitTask will be
     // fired on watcher thread).
     base::MessageLoop::current()->Run();
@@ skipping 29 matching lines @@
 }
 
 // Test verifies that \003 message send to process is processed as SigInt.
 // Timing out on the waterfall: http://crbug.com/115064
 TEST_F(ProcessProxyTest, DISABLED_SigInt) {
   test_runner_.reset(new SigIntTestRunner());
   RunTest();
 }
 
 }  // namespace chromeos