Don't use base::MessageLoop::{Quit,QuitClosure} in remoting/

remoting/protocol/authenticator_test_base.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/authenticator_test_base.h"
 
 #include "base/base64.h"
 #include "base/files/file_path.h"
 #include "base/files/file_util.h"
 #include "base/test/test_timeouts.h"
@@ skipping 13 matching lines @@
 
 namespace remoting {
 namespace protocol {
 
 namespace {
 
 ACTION_P(QuitThreadOnCounter, counter) {
   --(*counter);
   EXPECT_GE(*counter, 0);
   if (*counter == 0)
-    base::MessageLoop::current()->Quit();
+    base::MessageLoop::current()->QuitWhenIdle();
 }
 
 }  // namespace
 
 AuthenticatorTestBase::MockChannelDoneCallback::MockChannelDoneCallback() {}
 
 AuthenticatorTestBase::MockChannelDoneCallback::~MockChannelDoneCallback() {}
 
 AuthenticatorTestBase::AuthenticatorTestBase() {}
 
@@ skipping 90 matching lines @@
     EXPECT_CALL(host_callback_, OnDone(net::ERR_FAILED))
          .WillOnce(QuitThreadOnCounter(&callback_counter));
   } else {
     EXPECT_CALL(host_callback_, OnDone(net::OK))
         .WillOnce(QuitThreadOnCounter(&callback_counter));
   }
 
   // Ensure that .Run() does not run unbounded if the callbacks are never
   // called.
   base::Timer shutdown_timer(false, false);
-  shutdown_timer.Start(FROM_HERE,
-                       TestTimeouts::action_timeout(),
-                       base::MessageLoop::QuitClosure());
+  shutdown_timer.Start(FROM_HERE, TestTimeouts::action_timeout(),
+                       base::MessageLoop::QuitWhenIdleClosure());
   message_loop_.Run();
   shutdown_timer.Stop();
 
   testing::Mock::VerifyAndClearExpectations(&client_callback_);
   testing::Mock::VerifyAndClearExpectations(&host_callback_);
 
   if (!expected_fail) {
     ASSERT_TRUE(client_socket_.get() != nullptr);
     ASSERT_TRUE(host_socket_.get() != nullptr);
   }
 }
 
 void AuthenticatorTestBase::OnHostConnected(
     int error,
     scoped_ptr<P2PStreamSocket> socket) {
   host_callback_.OnDone(error);
   host_socket_ = socket.Pass();
 }
 
 void AuthenticatorTestBase::OnClientConnected(
     int error,
     scoped_ptr<P2PStreamSocket> socket) {
   client_callback_.OnDone(error);
   client_socket_ = socket.Pass();
 }
 
 }  // namespace protocol
 }  // namespace remoting