Adding the base ChromotingInstance implementation and unittests. This class will be used by the ap…

remoting/client/chromoting_client.cc

Index: remoting/client/chromoting_client.cc
diff --git a/remoting/client/chromoting_client.cc b/remoting/client/chromoting_client.cc
index 1ad7223f719f95c8096e2e23a46741e684f1d6ce..f250c82e714381864e67e09afeca1083a6e56645 100644
--- a/remoting/client/chromoting_client.cc
+++ b/remoting/client/chromoting_client.cc
@@ -33,6 +33,7 @@ ChromotingClient::ChromotingClient(
       user_interface_(user_interface),
       video_renderer_(video_renderer),
       host_capabilities_received_(false) {
+  connection_.reset(new protocol::ConnectionToHost());

[Sergey Ulanov, 2015/03/09 06:44:40]

this should be in the initializer list above, but I think it's better to move it
to Start().

[joedow, 2015/03/09 21:09:25]

The problem with moving this code to start is that the user of the class may
want to call SetProtocolConfigForTests() before calling Start() and then
connection_ will not be defined.  I've added it to the initialized list as I
think that is the safest solution and creating a ConnectionToHost that will be
overwritten by some tests seems like a reasonable trade-off.

   if (audio_player) {
     audio_decode_scheduler_.reset(new AudioDecodeScheduler(
         client_context->main_task_runner(),
@@ -45,7 +46,12 @@ ChromotingClient::~ChromotingClient() {}
 
 void ChromotingClient::SetProtocolConfigForTests(
     scoped_ptr<protocol::CandidateSessionConfig> config) {
-  connection_.set_candidate_config(config.Pass());
+  connection_->set_candidate_config(config.Pass());
+}
+
+void ChromotingClient::SetConnectionToHostForTests(
+    scoped_ptr<protocol::ConnectionToHost> connection_to_host) {
+  connection_ = connection_to_host.Pass();
 }
 
 void ChromotingClient::Start(
@@ -58,12 +64,12 @@ void ChromotingClient::Start(
 
   local_capabilities_ = capabilities;
 
-  connection_.set_client_stub(this);
-  connection_.set_clipboard_stub(this);
-  connection_.set_video_stub(video_renderer_->GetVideoStub());
-  connection_.set_audio_stub(audio_decode_scheduler_.get());
+  connection_->set_client_stub(this);
+  connection_->set_clipboard_stub(this);
+  connection_->set_video_stub(video_renderer_->GetVideoStub());
+  connection_->set_audio_stub(audio_decode_scheduler_.get());
 
-  connection_.Connect(signal_strategy, transport_factory.Pass(),
+  connection_->Connect(signal_strategy, transport_factory.Pass(),
                        authenticator.Pass(), host_jid, this);
 }
 
@@ -147,10 +153,9 @@ void ChromotingClient::OnAuthenticated() {
   DCHECK(task_runner_->BelongsToCurrentThread());
 
   // Initialize the decoder.
-  video_renderer_->OnSessionConfig(connection_.config());
-  if (connection_.config().is_audio_enabled())
-    audio_decode_scheduler_->Initialize(connection_.config());
-
+  video_renderer_->OnSessionConfig(connection_->config());
+  if (connection_->config().is_audio_enabled())
+    audio_decode_scheduler_->Initialize(connection_->config());
 }
 
 void ChromotingClient::OnChannelsConnected() {
@@ -161,7 +166,7 @@ void ChromotingClient::OnChannelsConnected() {
 
   protocol::Capabilities capabilities;
   capabilities.set_capabilities(local_capabilities_);
-  connection_.host_stub()->SetCapabilities(capabilities);
+  connection_->host_stub()->SetCapabilities(capabilities);
 }
 
 }  // namespace remoting