[Chromoting] Make ChromotingHost's dependency on libjingle injected, instead of hard-coded.

remoting/host/chromoting_host_unittest.cc

Index: remoting/host/chromoting_host_unittest.cc
diff --git a/remoting/host/chromoting_host_unittest.cc b/remoting/host/chromoting_host_unittest.cc
index b2b1bae8b65a137d3cd3bea356f56f8efab6148a..0f7637446e4a4df038166fb0fe4d072d4157f87d 100644
--- a/remoting/host/chromoting_host_unittest.cc
+++ b/remoting/host/chromoting_host_unittest.cc
@@ -92,9 +92,12 @@ class ChromotingHostTest : public testing::Test {
         &context_,
         capturer.Pass(),
         event_executor.Pass());
+    scoped_ptr<SessionManagerFactory> session_manager_factory(
+        new MockSessionManagerFactory());
 
     host_ = new ChromotingHost(
         &context_, &signal_strategy_, desktop_environment_.get(),
+        session_manager_factory.Pass(),

[Wez, 2012/06/11 22:17:06]

Do you need to assign the Mock to a temporary, above? Can't you just pass in the
new object directly here?

[simonmorris, 2012/06/11 23:08:59]

The mock will need more setting up when the unit tests are re-enabled. It'll be
clearer to spread the code out, and it'll be a smaller change if I leave it this
way for now.

         NetworkSettings());
 
     disconnect_window_ = new MockDisconnectWindow();