Refactor IT2Me-specific functions into a HostObserver subclass.

remoting/host/chromoting_host.cc

Index: remoting/host/chromoting_host.cc
diff --git a/remoting/host/chromoting_host.cc b/remoting/host/chromoting_host.cc
index c6485779d4b0a86aad1e02217653900f07e8b837..2c0cc1d5ee64a8e0a620890455d43d41e6e00557 100644
--- a/remoting/host/chromoting_host.cc
+++ b/remoting/host/chromoting_host.cc
@@ -14,7 +14,6 @@
 #include "remoting/base/encoder_row_based.h"
 #include "remoting/base/encoder_vp8.h"
 #include "remoting/host/chromoting_host_context.h"
-#include "remoting/host/curtain.h"
 #include "remoting/host/desktop_environment.h"
 #include "remoting/host/event_executor.h"
 #include "remoting/host/host_config.h"
@@ -57,7 +56,6 @@ ChromotingHost::ChromotingHost(ChromotingHostContext* context,
       stopping_recorders_(0),
       state_(kInitial),
       protocol_config_(protocol::CandidateSessionConfig::CreateDefault()),
-      is_curtained_(false),
       is_it2me_(false) {
   DCHECK(desktop_environment_);
   desktop_environment_->set_host(this);
@@ -200,12 +198,6 @@ void ChromotingHost::OnSessionAuthenticated(ClientSession* client) {
        it != status_observers_.end(); ++it) {
     (*it)->OnClientAuthenticated(jid);
   }
-  // TODO(jamiewalch): Tidy up actions to be taken on connect/disconnect,
-  // including closing the connection on failure of a critical operation.
-  EnableCurtainMode(true);
-
-  std::string username = jid.substr(0, jid.find('/'));
-  desktop_environment_->OnConnect(username);
 }
 
 void ChromotingHost::OnSessionClosed(ClientSession* client) {
@@ -231,10 +223,6 @@ void ChromotingHost::OnSessionClosed(ClientSession* client) {
       // Stop the recorder if there are no more clients.
       StopScreenRecorder();
     }
-
-    // Disable the "curtain" if there are no more active clients.
-    EnableCurtainMode(false);
-    desktop_environment_->OnLastDisconnect();
   }
 }
 
@@ -393,7 +381,6 @@ void ChromotingHost::PauseSession(bool pause) {
   for (client = clients_.begin(); client != clients_.end(); ++client) {
     (*client)->set_awaiting_continue_approval(pause);
   }
-  desktop_environment_->OnPause(pause);
 }
 
 void ChromotingHost::SetUiStrings(const UiStrings& ui_strings) {
@@ -436,16 +423,6 @@ int ChromotingHost::AuthenticatedClientsCount() const {
   return authenticated_clients;
 }
 
-void ChromotingHost::EnableCurtainMode(bool enable) {
-  // TODO(jamiewalch): This will need to be more sophisticated when we think
-  // about proper crash recovery and daemon mode.
-  // TODO(wez): CurtainMode shouldn't be driven directly by ChromotingHost.
-  if (is_it2me_ || enable == is_curtained_)
-    return;
-  desktop_environment_->curtain()->EnableCurtainMode(enable);
-  is_curtained_ = enable;
-}
-
 void ChromotingHost::StopScreenRecorder() {
   DCHECK(context_->network_message_loop()->BelongsToCurrentThread());
   DCHECK(recorder_.get());