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

remoting/host/host_mock_objects.h

 // Copyright (c) 2012 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_HOST_HOST_MOCK_OBJECTS_H_
 #define REMOTING_HOST_HOST_MOCK_OBJECTS_H_
 
 #include "net/base/ip_endpoint.h"
 #include "remoting/host/capturer.h"
 #include "remoting/host/curtain.h"
 #include "remoting/host/chromoting_host_context.h"
 #include "remoting/host/client_session.h"
 #include "remoting/host/continue_window.h"
 #include "remoting/host/disconnect_window.h"
 #include "remoting/host/event_executor.h"
 #include "remoting/host/local_input_monitor.h"
+#include "remoting/host/session_manager_factory.h"
 #include "remoting/host/user_authenticator.h"
 #include "testing/gmock/include/gmock/gmock.h"
 
 namespace remoting {
 
 class MockCapturer : public Capturer {
  public:
   MockCapturer();
   virtual ~MockCapturer();
 
@@ skipping 101 matching lines @@
   MOCK_METHOD1(InjectKeyEvent, void(const protocol::KeyEvent& event));
   MOCK_METHOD1(InjectMouseEvent, void(const protocol::MouseEvent& event));
   MOCK_METHOD0(OnSessionFinished, void());
 
   void OnSessionStarted(scoped_ptr<protocol::ClipboardStub> client_clipboard);
 
  private:
   DISALLOW_COPY_AND_ASSIGN(MockEventExecutor);
 };
 
+class MockSessionManagerFactory : public SessionManagerFactory {
+ public:
+  MockSessionManagerFactory();
+  virtual ~MockSessionManagerFactory();
+
+  MOCK_METHOD2(MakeSessionManagerPtr, protocol::SessionManager*(
+      const NetworkSettings& network_settings,
+      const scoped_refptr<net::URLRequestContextGetter>&
+          url_request_context_getter));
+  scoped_ptr<protocol::SessionManager> MakeSessionManager(
+      const NetworkSettings& network_settings,
+      const scoped_refptr<net::URLRequestContextGetter>&
+      url_request_context_getter) {
+    return scoped_ptr<protocol::SessionManager>(MakeSessionManagerPtr(
+        network_settings, url_request_context_getter));

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

Although it seems a bit pointless, I think this belongs in the .cc file...

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

This way is consistent with protocol_mock_objects.h, and I don't think it
matters in test code, especially when it's reducing boiler-plate so much.

+  }
+
+ private:
+  DISALLOW_COPY_AND_ASSIGN(MockSessionManagerFactory);
+};
+
 class MockUserAuthenticator : public UserAuthenticator {
  public:
   MockUserAuthenticator();
   virtual ~MockUserAuthenticator();
 
   MOCK_METHOD2(Authenticate, bool(const std::string& username,
                                   const std::string& password));
 
  private:
   DISALLOW_COPY_AND_ASSIGN(MockUserAuthenticator);
 };
 
 }  // namespace remoting
 
 #endif  // REMOTING_HOST_HOST_MOCK_OBJECTS_H_