A new authenticated connection evicts an old one.

remoting/protocol/host_stub.h

-// Copyright (c) 2010 The Chromium Authors. All rights reserved.
+// Copyright (c) 2011 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 host that receives commands from a Chromoting client.
 //
 // This interface handles control messages defined in contro.proto.
 
 #ifndef REMOTING_PROTOCOL_HOST_STUB_H_
 #define REMOTING_PROTOCOL_HOST_STUB_H_
 
 #include "base/basictypes.h"
 
 class Task;
 
 namespace remoting {
 namespace protocol {
 
+class ConnectionToClient;
+class LocalLoginCredentials;
 class SuggestResolutionRequest;
-class LocalLoginCredentials;
 
 class HostStub {
  public:
   HostStub();
   virtual ~HostStub();
 
   virtual void SuggestResolution(
       const SuggestResolutionRequest* msg, Task* done) = 0;
   virtual void BeginSessionRequest(
+      ConnectionToClient* connection,

[Sergey Ulanov, 2011/03/18 18:12:21]

Can we avoid adding this parameter? HostStub is supposed to be used on both side
of the connection, yet, ConnectionToClient doesn't make sense on the client
side.

[simonmorris, 2011/03/22 13:09:10]

Done.

       const LocalLoginCredentials* credentials, Task* done) = 0;
 
   // TODO(lambroslambrou): Remove OnAuthenticated() and OnClosed() when stubs
   // are refactored not to store authentication state.
 
   // Called when the client has authenticated with the host to enable the
   // client->host control channel.
   // Before this is called, only a limited set of control messages will be
   // processed.
   void OnAuthenticated();
@@ skipping 10 matching lines @@
   // the host.
   bool authenticated_;
 
   DISALLOW_COPY_AND_ASSIGN(HostStub);
 };
 
 }  // namespace protocol
 }  // namespace remoting
 
 #endif  // REMOTING_PROTOCOL_HOST_STUB_H_