A new authenticated connection evicts an old one.

remoting/protocol/connection_to_client.h

Index: remoting/protocol/connection_to_client.h
diff --git a/remoting/protocol/connection_to_client.h b/remoting/protocol/connection_to_client.h
index d2721f3303c2c34689882b27128e487502b55685..2cca222e2f1c6137ebf23b3352fa058e825eb8df 100644
--- a/remoting/protocol/connection_to_client.h
+++ b/remoting/protocol/connection_to_client.h
@@ -1,4 +1,4 @@
-// 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.
 
@@ -53,8 +53,6 @@ class ConnectionToClient :
                      HostStub* host_stub,
                      InputStub* input_stub);
 
-  virtual ~ConnectionToClient();
-
   virtual void Init(Session* session);
 
   // Returns the connection in use.
@@ -72,9 +70,19 @@ class ConnectionToClient :
   // Return pointer to ClientStub.
   virtual ClientStub* client_stub();
 
+  virtual HostStub* host_stub();
+  virtual void set_host_stub(HostStub* host_stub);
+
   // Called when the host accepts the client authentication.
   void OnClientAuthenticated();
 
+  // Whether the client has been authenticated.
+  bool client_authenticated();
+
+ protected:
+  friend class base::RefCountedThreadSafe<ConnectionToClient>;
+  virtual ~ConnectionToClient();

[awong, 2011/03/22 15:11:56]

Does this need to be virtual?

[Sergey Ulanov, 2011/03/22 23:19:14]

I think the rule is to make destructor virtual if there is at least one virtual
method.

http://google-styleguide.googlecode.com/svn/trunk/cppguide.xml?showone=Inheri...

[simonmorris, 2011/03/23 10:35:20]

Yes: this is a base class for MockConnectionToClient.

+
  private:
   // Callback for protocol Session.
   void OnSessionStateChange(Session::State state);