A new authenticated connection evicts an old one.

remoting/protocol/connection_to_client.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.
 
 #ifndef REMOTING_PROTOCOL_CONNECTION_TO_CLIENT_H_
 #define REMOTING_PROTOCOL_CONNECTION_TO_CLIENT_H_
 
 #include <deque>
 #include <vector>
 
 #include "base/message_loop.h"
@@ skipping 34 matching lines @@
 
   // Constructs a ConnectionToClient object. |message_loop| is the message loop
   // that this object runs on. A viewer object receives events and messages from
   // a libjingle channel, these events are delegated to |handler|.
   // It is guranteed that |handler| is called only on the |message_loop|.
   ConnectionToClient(MessageLoop* message_loop,
                      EventHandler* handler,
                      HostStub* host_stub,
                      InputStub* input_stub);
 
-  virtual ~ConnectionToClient();
-
   virtual void Init(Session* session);
 
   // Returns the connection in use.
   virtual Session* session();
 
   // Disconnect the client connection. This method is allowed to be called
   // more than once and calls after the first one will be ignored.
   //
   // 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();
 
+  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);
 
   // 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
@@ skipping 24 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_