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

remoting/host/session_manager_factory.h

Index: remoting/host/session_manager_factory.h
diff --git a/remoting/host/session_manager_factory.h b/remoting/host/session_manager_factory.h
new file mode 100644
index 0000000000000000000000000000000000000000..432db89b16cef807f16d40e47778740a3583ac70
--- /dev/null
+++ b/remoting/host/session_manager_factory.h
@@ -0,0 +1,31 @@
+// 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_SESSION_MANAGER_FACTORY_H_
+#define REMOTING_HOST_SESSION_MANAGER_FACTORY_H_

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

Since this is a factory for remoting::protocol::SessionManager, perhaps it would
make more sense under remoting/protocol/?  The interface could even be defined
in remoting/protocol/session_manager.h to save on a bit of boilerplate, if you
wanted.

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

That's where I put it to begin with, but NetworkSettings is in remoting/host,
and protocol_mock_objects.h would have had to include it.

+
+#include "base/memory/scoped_ptr.h"
+#include "remoting/protocol/session_manager.h"
+
+namespace net {
+class URLRequestContextGetter;
+}  // namespace net
+
+namespace remoting {
+
+struct NetworkSettings;
+
+class SessionManagerFactory {
+ public:
+  virtual ~SessionManagerFactory() {};
+
+  virtual scoped_ptr<protocol::SessionManager> MakeSessionManager(
+      const NetworkSettings& network_settings,
+      const scoped_refptr<net::URLRequestContextGetter>&
+          url_request_context_getter) = 0;
+};
+
+}  // namespace remoting
+
+#endif  // REMOTING_HOST_SESSION_MANAGER_FACTORY_H_