Chromoting protocol layers to receive and send login messages

remoting/protocol/jingle_connection_to_host.cc

 // Copyright (c) 2010 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.
 
 #include "remoting/protocol/jingle_connection_to_host.h"
 
 #include "base/callback.h"
 #include "base/message_loop.h"
 #include "remoting/base/constants.h"
 #include "remoting/jingle_glue/jingle_thread.h"
 #include "remoting/proto/auth.pb.h"
+#include "remoting/protocol/client_message_dispatcher.h"
 #include "remoting/protocol/client_stub.h"
 #include "remoting/protocol/input_sender.h"
 #include "remoting/protocol/jingle_session_manager.h"
 #include "remoting/protocol/video_reader.h"
 #include "remoting/protocol/video_stub.h"
 #include "remoting/protocol/util.h"
 
 namespace remoting {
 namespace protocol {
 
 JingleConnectionToHost::JingleConnectionToHost(JingleThread* thread)
     : thread_(thread),
-      event_callback_(NULL) {
+      event_callback_(NULL),
+      dispatcher_(new ClientMessageDispatcher()) {
 }
 
 JingleConnectionToHost::~JingleConnectionToHost() {
 }
 
 void JingleConnectionToHost::Connect(const std::string& username,
                                      const std::string& auth_token,
                                      const std::string& host_jid,
                                      HostEventCallback* event_callback,
                                      ClientStub* client_stub,
                                      VideoStub* video_stub) {
   event_callback_ = event_callback;
+  client_stub_ = client_stub;
   video_stub_ = video_stub;
 
   // Initialize |jingle_client_|.
   jingle_client_ = new JingleClient(thread_);
   jingle_client_->Init(username, auth_token,
                        kChromotingTokenServiceName, this);
 
   // Save jid of the host. The actual connection is created later after
   // |jingle_client_| is connected.
   host_jid_ = host_jid;
@@ skipping 21 matching lines @@
 void JingleConnectionToHost::OnControlMessage(ControlMessage* msg) {
   // TODO(sergeyu): Remove this method and pass ClientStub to the control
   // stream dispatcher.
   delete msg;
 }
 
 void JingleConnectionToHost::InitSession() {
   DCHECK_EQ(message_loop(), MessageLoop::current());
 
   // Initialize chromotocol |session_manager_|.
-  protocol::JingleSessionManager* session_manager =
-      new protocol::JingleSessionManager(thread_);
+  JingleSessionManager* session_manager =
+      new JingleSessionManager(thread_);
   // TODO(ajwong): Make this a command switch when we're more stable.
   session_manager->set_allow_local_ips(true);
   session_manager->Init(
       jingle_client_->GetFullJid(),
       jingle_client_->session_manager(),
       NewCallback(this, &JingleConnectionToHost::OnNewSession));
   session_manager_ = session_manager;
 
   CandidateSessionConfig* candidate_config =
       CandidateSessionConfig::CreateDefault();
@@ skipping 47 matching lines @@
 
   if (state == JingleClient::CONNECTED) {
     VLOG(1) << "Connected as: " << client->GetFullJid();
     InitSession();
   } else if (state == JingleClient::CLOSED) {
     VLOG(1) << "Connection closed.";
     event_callback_->OnConnectionClosed(this);
   }
 }
 
-void JingleConnectionToHost::OnNewSession(protocol::Session* session,
-    protocol::SessionManager::IncomingSessionResponse* response) {
+void JingleConnectionToHost::OnNewSession(Session* session,
+    SessionManager::IncomingSessionResponse* response) {
   DCHECK_EQ(message_loop(), MessageLoop::current());
   // Client always rejects incoming sessions.
-  *response = protocol::SessionManager::DECLINE;
+  *response = SessionManager::DECLINE;
 }
 
 void JingleConnectionToHost::OnSessionStateChange(
-    protocol::Session::State state) {
+    Session::State state) {
   DCHECK_EQ(message_loop(), MessageLoop::current());
   DCHECK(event_callback_);
 
   switch (state) {
-    case protocol::Session::FAILED:
+    case Session::FAILED:
       event_callback_->OnConnectionFailed(this);
       break;
 
-    case protocol::Session::CLOSED:
+    case Session::CLOSED:
       event_callback_->OnConnectionClosed(this);
       break;
 
-    case protocol::Session::CONNECTED:
+    case Session::CONNECTED:
       // Initialize reader and writer.
       control_reader_.Init<ControlMessage>(
           session_->control_channel(),
           NewCallback(this, &JingleConnectionToHost::OnControlMessage));
       video_reader_.reset(VideoReader::Create(session_->config()));
       video_reader_->Init(session_, video_stub_);
       input_stub_.reset(new InputSender(session_->event_channel()));
+      dispatcher_->Initialize(session_.get(), client_stub_);

[awong, 2010/12/23 01:15:10]

If Initialize() reutrns a bool, shouldn't we check it here?

[Alpha Left Google, 2010/12/23 02:17:58]

Done.

       event_callback_->OnConnectionOpened(this);
       break;
 
     default:
       // Ignore the other states by default.
       break;
   }
 }
 
 MessageLoop* JingleConnectionToHost::message_loop() {
   return thread_->message_loop();
 }
 
 }  // namespace protocol
 }  // namespace remoting