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

remoting/protocol/client_stub.h

Index: remoting/protocol/client_stub.h
diff --git a/remoting/protocol/client_stub.h b/remoting/protocol/client_stub.h
index 4b73789b84abeb95623275e5bee259665e2447b8..569d96eb331ca968ed1a56b4937b72140ab26bcd 100644
--- a/remoting/protocol/client_stub.h
+++ b/remoting/protocol/client_stub.h
@@ -22,15 +22,30 @@ class NotifyResolutionRequest;
 
 class ClientStub {
  public:
-  ClientStub() {}
-  virtual ~ClientStub() {}
+  ClientStub();

[Wez, 2011/03/04 12:09:38]

Is there a more descriptive name we can attached to this, and make it more
uniform with the related Input and Video classes? e.g. ClientControl,
ClientInput, ClientVideo

[garykac, 2011/03/04 20:28:02]

I agree that it would be nice to have better names, but I'll defer that to a
separate CL.

+  virtual ~ClientStub();
 
   virtual void NotifyResolution(const NotifyResolutionRequest* msg,
                                 Task* done) = 0;
   virtual void BeginSessionResponse(const LocalLoginStatus* msg,
                                     Task* done) = 0;
 
+  // Enable the host->client control channel.
+  // This should be set to true only after a trusted communication channel has
+  // been established.
+  // When this is false, only a limited set of control messages will be
+  // processed.
+  void SetEnabled(bool enabled);
+

[Wez, 2011/03/04 12:09:38]

Move from SetEnabled(bool) to OnAuthenticated()?

[garykac, 2011/03/04 20:28:02]

Done.

+  // Is the control channel enabled?
+  // I.e., should we be processing control events?
+  bool enabled();

[Wez, 2011/03/04 12:09:38]

Do we need this?

[garykac, 2011/03/04 20:28:02]

The message dispatchers make use of this to cut off processing the input or
control packets.  These dispatchers don't currently have access to the
connection_to_host/client, so it's convenient to have it here.

+
  private:
+  // Initially false, this controls whether the control events received from
+  // the host should be processed.
+  bool enabled_;
+
   DISALLOW_COPY_AND_ASSIGN(ClientStub);
 };