Handle disconnection events properly for unauthenicated clients.

remoting/host/client_session.h

Index: remoting/host/client_session.h
diff --git a/remoting/host/client_session.h b/remoting/host/client_session.h
index 084d153a2de2acaa64684c73dc115e6ed6375d30..e61e84604dbd5e3d6c3546fd8c0812811b20cae3 100644
--- a/remoting/host/client_session.h
+++ b/remoting/host/client_session.h
@@ -109,6 +109,8 @@ class ClientSession : public protocol::HostEventStub,
 
   const std::string& client_jid() { return client_jid_; }
 
+  bool is_authenticated() { return is_authenticated_;  }

[Wez, 2012/05/18 20:20:57]

You can check |auth_input_filter_.get_stub()| here and avoid having a separate
|is_authenticated_| flag.

[Sergey Ulanov, 2012/05/21 18:20:13]

OnConnectionClosed() resets auth_input_filter_ and we need this method to return
correct value when OnSessionClosed() is called.

[Wez, 2012/05/21 20:33:50]

Hmmm, yes; the teardown logic in ChromotingHost really belongs in ClientSession
- ClientSession should just tell the host when it needs the ScreenRecorder, and
when it's done with it.

[Sergey Ulanov, 2012/05/21 22:27:55]

Added TODO.

+
   // Indicate that local mouse activity has been detected. This causes remote
   // inputs to be ignored for a short time so that the local user will always
   // have the upper hand in 'pointer wars'.
@@ -125,6 +127,7 @@ class ClientSession : public protocol::HostEventStub,
   scoped_ptr<protocol::ConnectionToClient> connection_;
 
   std::string client_jid_;
+  bool is_authenticated_;
 
   // The host event stub to which this object delegates. This is the final
   // element in the input pipeline, whose components appear in order below.