Making DesktopEnvironment a factory class used by ClientSession to create audio/video capturers and…

remoting/host/host_mock_objects.cc

 // 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.
 
 #include "remoting/host/host_mock_objects.h"
 
 #include "base/message_loop_proxy.h"
+#include "base/single_thread_task_runner.h"
 #include "net/base/ip_endpoint.h"
 #include "remoting/base/auto_thread_task_runner.h"
+#include "remoting/capturer/video_frame_capturer.h"
+#include "remoting/codec/audio_encoder.h"
+#include "remoting/codec/video_encoder.h"
+#include "remoting/host/audio_capturer.h"
+#include "remoting/host/event_executor.h"
 #include "remoting/proto/event.pb.h"
 #include "remoting/protocol/transport.h"
 
 namespace remoting {
 
-MockDesktopEnvironmentFactory::MockDesktopEnvironmentFactory()
-    : DesktopEnvironmentFactory(NULL, NULL) {
+MockDesktopEnvironment::MockDesktopEnvironment() {}
+
+MockDesktopEnvironment::~MockDesktopEnvironment() {}
+
+scoped_ptr<AudioCapturer> MockDesktopEnvironment::CreateAudioCapturer(
+    scoped_refptr<base::SingleThreadTaskRunner> audio_task_runner) {
+  return scoped_ptr<AudioCapturer>(CreateAudioCapturerPtr(audio_task_runner));

[Wez, 2013/01/11 02:23:41]

audio_task_runner.get() - implicit cast to raw pointer is going away, IIRC.

[alexeypa (please no reviews), 2013/01/11 19:59:46]

CreateAudioCapturerPtr takes scoped_refptr. The return type is the reason why
*Ptr wrappers are needed.

[Wez, 2013/01/12 01:58:06]

Ah yes!

+}
+
+scoped_ptr<EventExecutor> MockDesktopEnvironment::CreateEventExecutor(
+    scoped_refptr<base::SingleThreadTaskRunner> input_task_runner,
+    scoped_refptr<base::SingleThreadTaskRunner> ui_task_runner) {
+  return scoped_ptr<EventExecutor>(CreateEventExecutorPtr(input_task_runner,
+                                                          ui_task_runner));

[Wez, 2013/01/11 02:23:41]

Ditto.

[alexeypa (please no reviews), 2013/01/11 19:59:46]

See above.

+}
+
+scoped_ptr<VideoFrameCapturer> MockDesktopEnvironment::CreateVideoCapturer(
+    scoped_refptr<base::SingleThreadTaskRunner> capture_task_runner,
+    scoped_refptr<base::SingleThreadTaskRunner> encode_task_runner) {
+  return scoped_ptr<VideoFrameCapturer>(CreateVideoCapturerPtr(
+      capture_task_runner, encode_task_runner));

[Wez, 2013/01/11 02:23:41]

Here too.

[alexeypa (please no reviews), 2013/01/11 19:59:46]

See above.

+}
+
+MockDesktopEnvironmentFactory::MockDesktopEnvironmentFactory(
+    scoped_refptr<base::SingleThreadTaskRunner> caller_task_runner)
+    : DesktopEnvironmentFactory(caller_task_runner) {
 }
 
 MockDesktopEnvironmentFactory::~MockDesktopEnvironmentFactory() {}
 
-scoped_ptr<DesktopEnvironment> MockDesktopEnvironmentFactory::Create() {
+scoped_ptr<DesktopEnvironment> MockDesktopEnvironmentFactory::Create(
+    const std::string& client_jid,
+    const base::Closure& disconnect_callback) {
   return scoped_ptr<DesktopEnvironment>(CreatePtr());
 }
 
 MockEventExecutor::MockEventExecutor() {}
 
 MockEventExecutor::~MockEventExecutor() {}
 
 void MockEventExecutor::Start(
     scoped_ptr<protocol::ClipboardStub> client_clipboard) {
   StartPtr(client_clipboard.get());
@@ skipping 21 matching lines @@
 
 MockClientSessionEventHandler::MockClientSessionEventHandler() {}
 
 MockClientSessionEventHandler::~MockClientSessionEventHandler() {}
 
 MockHostStatusObserver::MockHostStatusObserver() {}
 
 MockHostStatusObserver::~MockHostStatusObserver() {}
 
 }  // namespace remoting