Move ConnectionToClient::EventHandler from ChromotingHost to ClientSession

remoting/protocol/connection_to_client.h

 // 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>
 
@@ skipping 38 matching lines @@
     virtual void OnConnectionFailed(ConnectionToClient* connection) = 0;
 
     // Called when sequence number is updated.
     virtual void OnSequenceNumberUpdated(ConnectionToClient* connection,
                                          int64 sequence_number) = 0;
   };
 
   // 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 guaranteed that |handler| is called only on the |message_loop|.

[Wez, 2011/11/09 01:35:07]

Update this comment now that the function signature has changed.

[Sergey Ulanov, 2011/11/09 19:26:13]

Done.

-  ConnectionToClient(base::MessageLoopProxy* message_loop,
-                     EventHandler* handler);
+  ConnectionToClient(base::MessageLoopProxy* message_loop, Session* session);
 
-  virtual void Init(Session* session);
+  void SetEventHandler(EventHandler* event_handler);

[Wez, 2011/11/09 01:35:07]

Clarify the semantics?  What happens if events happen before I set a handler? 
Can I change the handler mid-session?  Can I clear the handler?

[Sergey Ulanov, 2011/11/09 19:26:13]

Done.

 
   // 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();
 
@@ skipping 54 matching lines @@
   bool input_connected_;
   bool video_connected_;
 
   DISALLOW_COPY_AND_ASSIGN(ConnectionToClient);
 };
 
 }  // namespace protocol
 }  // namespace remoting
 
 #endif  // REMOTING_PROTOCOL_CONNECTION_TO_CLIENT_H_