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

remoting/protocol/client_stub.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.
 
 // Interface of a client that receives commands from a Chromoting host.
 //
 // This interface is responsible for a subset of control messages sent to
 // the Chromoting client.
 
 #ifndef REMOTING_PROTOCOL_CLIENT_STUB_H_
 #define REMOTING_PROTOCOL_CLIENT_STUB_H_
 
 #include "base/basictypes.h"
 
 class Task;
 
 namespace remoting {
 namespace protocol {
 
 class LocalLoginStatus;
 class NotifyResolutionRequest;
 
 class ClientStub {
  public:
-  ClientStub() {}
+  ClientStub() : enable_(false) {}
   virtual ~ClientStub() {}

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

as this is no longer pure virtual, may have to move the d'tor out of line

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

Done.

 
   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 SetEnable(bool enable) {

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

actually the var name should probably be enabled, and this should be
set_enabled(bool enabled).

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

Done.

+    enable_ = enable;
+  }
+
+  // Is the control channel enabled?
+  // I.e., should we be processing control events?
+  bool Enabled() {

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

just enabled()

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

SetEnabled() and enabled()?

or would set_enabled() be better?

+    return enable_;
+  }
+
  private:
+  // Initially false, this controls whether the control events received from the
+  // host should be processed.
+  bool enable_;
+
   DISALLOW_COPY_AND_ASSIGN(ClientStub);
 };
 
 }  // namespace protocol
 }  // namespace remoting
 
 #endif  // REMOTING_PROTOCOL_CLIENT_STUB_H_