Block event processing on host/client until the client has authenticated.

remoting/protocol/connection_to_client.h

 // Copyright (c) 2010 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.
 
 #ifndef REMOTING_PROTOCOL_CONNECTION_TO_CLIENT_H_
 #define REMOTING_PROTOCOL_CONNECTION_TO_CLIENT_H_
 
 #include <deque>
 #include <vector>
 
@@ skipping 54 matching lines @@
   //
   // After this method is called all the send method calls will be ignored.
   virtual void Disconnect();
 
   // Send encoded update stream data to the viewer.
   virtual VideoStub* video_stub();
 
   // Return pointer to ClientStub.
   virtual ClientStub* client_stub();
 
+  // Record whether the client has been properly authenticated with the host.
+  void SetClientAuthenticated(bool auth);
+
  protected:
   // Protected constructor used by unit test.
-  ConnectionToClient();
+  //ConnectionToClient();

[dmac, 2011/03/04 01:36:27]

should we get rid of this then?

[garykac, 2011/03/04 05:33:24]

oops

 
  private:
   // Callback for protocol Session.
   void OnSessionStateChange(Session::State state);
 
   // Process a libjingle state change event on the |loop_|.
   void StateChangeTask(Session::State state);
 
   void OnClosed();
 
+  // Initially false, this is set to true once the client has authenticated
+  // properly. When this is false, many client messages (like input events)
+  // will be silently ignored.

[dmac, 2011/03/04 01:36:27]

will they be silently ignored? or will LOGs occur?

[garykac, 2011/03/04 05:33:24]

They should LOG, so I removed "silently"

+  bool client_authenticated_;
+
   // The libjingle channel used to send and receive data from the remote client.
   scoped_refptr<Session> session_;
 
   scoped_ptr<VideoWriter> video_writer_;
 
   // ClientStub for sending messages to the client.
   scoped_ptr<ClientStub> client_stub_;
 
   // The message loop that this object runs on.
   MessageLoop* loop_;
@@ skipping 10 matching lines @@
   // Dispatcher for submitting messages to stubs.
   scoped_ptr<HostMessageDispatcher> dispatcher_;
 
   DISALLOW_COPY_AND_ASSIGN(ConnectionToClient);
 };
 
 }  // namespace protocol
 }  // namespace remoting
 
 #endif  // REMOTING_PROTOCOL_CONNECTION_TO_CLIENT_H_