Fix ConnectionToClient to connect stubs only after authentication.

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 a70fd3b5c5fb42d78216630a35557fc5a6a93a54..57fa401164bc1838ace4a1fd335585bbc313cd93 100644
--- a/remoting/host/client_session.h
+++ b/remoting/host/client_session.h
@@ -141,7 +141,7 @@ class ClientSession
     return connection_.get();
   }
 
-  bool is_authenticated() { return auth_input_filter_.enabled();  }
+  bool is_authenticated() { return is_authenticated_;  }
 
   const std::string* client_capabilities() const {
     return client_capabilities_.get();
@@ -185,10 +185,6 @@ class ClientSession
   protocol::InputFilter disable_input_filter_;
   protocol::ClipboardFilter disable_clipboard_filter_;
 
-  // Filters used to disable input & clipboard when we're not authenticated.
-  protocol::InputFilter auth_input_filter_;
-  protocol::ClipboardFilter auth_clipboard_filter_;
-
   // Factory for weak pointers to the client clipboard stub.
   // This must appear after |clipboard_echo_filter_|, so that it won't outlive
   // it.
@@ -240,6 +236,8 @@ class ClientSession
   // Used to manage extension functionality.
   scoped_ptr<HostExtensionSessionManager> extension_manager_;
 
+  bool is_authenticated_;

[Wez, 2015/02/07 00:40:43]

nit: Add a comment to clarify that this is true if the connection has
successfully authenticated, so it's obvious that it's not a flag indicating
whether the connection _should_ bother doing authentication. :)

[Sergey Ulanov, 2015/02/13 19:40:21]

Done.

+
   // Used to store video channel pause & lossless parameters.
   bool pause_video_;
   bool lossless_video_encode_;