Remove video_channel() from Session interface

remoting/protocol/connection_to_host.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_HOST_H_
 #define REMOTING_PROTOCOL_CONNECTION_TO_HOST_H_
 
 #include <string>
 
 #include "base/memory/ref_counted.h"
@@ skipping 106 matching lines @@
   State state() const;
 
  private:
   // Called on the jingle thread after we've successfully to XMPP server. Starts
   // P2P connection to the host.
   void InitSession();
 
   // Callback for |session_|.
   void OnSessionStateChange(Session::State state);
 
+  // Callback for VideoReader::Init().
+  void OnVideoChannelInitialized(bool successful);
+
+  void NotifyIfChannelsReady();
+
   // Callback for |video_reader_|.
   void OnVideoPacket(VideoPacket* packet);
 
+  void CloseOnError();
+
   // Stops writing in the channels.
   void CloseChannels();
 
   MessageLoop* message_loop_;
   scoped_ptr<talk_base::NetworkManager> network_manager_;
   scoped_ptr<talk_base::PacketSocketFactory> socket_factory_;
   scoped_ptr<HostResolverFactory> host_resolver_factory_;
   scoped_ptr<PortAllocatorSessionFactory> port_allocator_session_factory_;
   bool allow_nat_traversal_;
 
-  // Internal state of the connection.
-  State state_;
+  std::string host_jid_;
+  std::string host_public_key_;
+  std::string access_code_;
+
+  HostEventCallback* event_callback_;
+
+  // Stub for incoming messages.
+  ClientStub* client_stub_;
+  VideoStub* video_stub_;
 
   scoped_ptr<SignalStrategy> signal_strategy_;
   std::string local_jid_;
   scoped_ptr<SessionManager> session_manager_;
   scoped_ptr<Session> session_;
 
+  // Handlers for incoming messages.
   scoped_ptr<VideoReader> video_reader_;
-
-  HostEventCallback* event_callback_;
-
-  std::string host_jid_;
-  std::string host_public_key_;
-  std::string access_code_;
-
   scoped_ptr<ClientMessageDispatcher> dispatcher_;
 
-  ////////////////////////////////////////////////////////////////////////////
-  // User input event channel interface
-
-  // Stub for sending input event messages to the host.
+  // Senders for outgoing messages.
   scoped_ptr<InputSender> input_sender_;
-
-  ////////////////////////////////////////////////////////////////////////////
-  // Protocol control channel interface
-
-  // Stub for sending control messages to the host.
   scoped_ptr<HostControlSender> host_control_sender_;
 
-  // Stub for receiving control messages from the host.
-  ClientStub* client_stub_;
+  // Internal state of the connection.
+  State state_;
 
-  ////////////////////////////////////////////////////////////////////////////
-  // Video channel interface
-
-  // Stub for receiving video packets from the host.
-  VideoStub* video_stub_;
+  // State of the channels.
+  bool control_connected_;
+  bool input_connected_;
+  bool video_connected_;
 
  private:
   DISALLOW_COPY_AND_ASSIGN(ConnectionToHost);
 };
 
 }  // namespace protocol
 }  // namespace remoting
 
 DISABLE_RUNNABLE_METHOD_REFCOUNT(remoting::protocol::ConnectionToHost);
 
 #endif  // REMOTING_PROTOCOL_CONNECTION_TO_HOST_H_