Simplify It2Me host shutdown

Previously It2Me host was stopped asynchronoulsy, which makes the
shutdown more complicated. Now it's synchronous and is significantly
simpler.
Also removed ChromotingHost::RejectAuthenticatingClient() which is no
longer needed.

Review URL: https://codereview.chromium.org/1458323002

Cr-Commit-Position: refs/heads/master@{#361362}

1 files changed, 0 insertions, 20 deletions

diff --git a/remoting/host/chromoting_host_unittest.cc b/remoting/host/chromoting_host_unittest.cc
index 861d116..cd54719 100644
--- a/remoting/host/chromoting_host_unittest.cc
+++ b/remoting/host/chromoting_host_unittest.cc
@@ -270,11 +270,6 @@ class ChromotingHostTest : public testing::Test {
     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(
@@ -468,21 +463,6 @@ TEST_F(ChromotingHostTest, Connect) {
   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_))