Simplify It2Me host shutdown

remoting/host/chromoting_host_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 "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/memory/scoped_ptr.h"
 #include "remoting/base/auto_thread_task_runner.h"
 #include "remoting/host/audio_capturer.h"
 #include "remoting/host/chromoting_host.h"
@@ skipping 252 matching lines @@
 
   void DisconnectAllClients() {
     host_->DisconnectAllClients();
   }
 
   // Helper method to disconnect client 1 from the host.
   void DisconnectClient1() {
     NotifyClientSessionClosed(0);
   }
 
-  // Notify |host_| that the authenticating client has been rejected.
-  void RejectAuthenticatingClient() {
-    host_->RejectAuthenticatingClient();
-  }
-
   // Notify |host_| that a client session has closed.
   void NotifyClientSessionClosed(int connection_index) {
     get_client(connection_index)->OnConnectionClosed(
         get_connection(connection_index), protocol::OK);
   }
 
   void NotifyConnectionClosed1() {
     if (session_unowned1_event_handler_) {
       session_unowned1_event_handler_->OnSessionStateChange(Session::CLOSED);
     }
@@ skipping 173 matching lines @@
       0, InvokeWithoutArgs(this, &ChromotingHostTest::ShutdownHost));
   Expectation client_disconnected = ExpectClientDisconnected(
       0, true, video_packet_sent, InvokeWithoutArgs(base::DoNothing));
   EXPECT_CALL(host_status_observer_, OnShutdown()).After(client_disconnected);
 
   host_->Start(xmpp_login_);
   SimulateClientConnection(0, true, false);
   message_loop_.Run();
 }
 
-TEST_F(ChromotingHostTest, RejectAuthenticatingClient) {
-  Expectation start = ExpectHostAndSessionManagerStart();
-  EXPECT_CALL(host_status_observer_, OnClientAuthenticated(session_jid1_))
-      .WillOnce(InvokeWithoutArgs(
-      this, &ChromotingHostTest::RejectAuthenticatingClient));
-  ExpectClientDisconnected(
-      0, true, start,
-      InvokeWithoutArgs(this, &ChromotingHostTest::ShutdownHost));
-  EXPECT_CALL(host_status_observer_, OnShutdown());
-
-  host_->Start(xmpp_login_);
-  SimulateClientConnection(0, true, true);
-  message_loop_.Run();
-}
-
 TEST_F(ChromotingHostTest, AuthenticationFailed) {
   ExpectHostAndSessionManagerStart();
   EXPECT_CALL(host_status_observer_, OnAccessDenied(session_jid1_))
       .WillOnce(InvokeWithoutArgs(this, &ChromotingHostTest::ShutdownHost));
   EXPECT_CALL(host_status_observer_, OnShutdown());
 
   host_->Start(xmpp_login_);
   SimulateClientConnection(0, false, false);
   message_loop_.Run();
 }
@@ skipping 162 matching lines @@
   ExpectClientDisconnected(0, true, video_packet_sent,
       InvokeWithoutArgs(this, &ChromotingHostTest::ShutdownHost));
   EXPECT_CALL(host_status_observer_, OnShutdown());
 
   host_->Start(xmpp_login_);
   SimulateClientConnection(0, true, false);
   message_loop_.Run();
 }
 
 }  // namespace remoting