[Chromoting] Refactoring DesktopEnvironment and moving screen/audio recorders to ClientSession.

remoting/host/desktop_environment.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_DESKTOP_ENVIRONMENT_H_
 #define REMOTING_HOST_DESKTOP_ENVIRONMENT_H_
 
-#include <string>
-
 #include "base/basictypes.h"
 #include "base/memory/scoped_ptr.h"
 
 namespace remoting {
 
 class AudioCapturer;
 class ChromotingHostContext;
 class EventExecutor;
 class VideoFrameCapturer;
 
 namespace protocol {
 class ClipboardStub;
 }
 
 class DesktopEnvironment {
  public:
-  // Creates a DesktopEnvironment used in a host plugin.
-  static scoped_ptr<DesktopEnvironment> Create(
-      ChromotingHostContext* context);
-
-  // Creates a DesktopEnvironment used in a service process.
-  static scoped_ptr<DesktopEnvironment> CreateForService(
-      ChromotingHostContext* context);
-
-  static scoped_ptr<DesktopEnvironment> CreateFake(
-      ChromotingHostContext* context,
-      scoped_ptr<VideoFrameCapturer> capturer,
-      scoped_ptr<EventExecutor> event_executor,
-      scoped_ptr<AudioCapturer> audio_capturer);
-
+  DesktopEnvironment(scoped_ptr<AudioCapturer> audio_capturer,
+                     scoped_ptr<EventExecutor> event_executor,
+                     scoped_ptr<VideoFrameCapturer> video_capturer);
   virtual ~DesktopEnvironment();
 
-  VideoFrameCapturer* capturer() const { return capturer_.get(); }
+  AudioCapturer* audio_capturer() const { return audio_capturer_.get(); }
   EventExecutor* event_executor() const { return event_executor_.get(); }
-  AudioCapturer* audio_capturer() const { return audio_capturer_.get(); }
-  void OnSessionStarted(scoped_ptr<protocol::ClipboardStub> client_clipboard);
-  void OnSessionFinished();
+  VideoFrameCapturer* video_capturer() const { return video_capturer_.get(); }
+
+  virtual void Start(
+      scoped_ptr<protocol::ClipboardStub> client_clipboard);
+  virtual void StopAndDelete();

[Wez, 2012/08/31 18:18:02]

We don't actually need StopAndDelete() here, since we could StopAndDelete() the
EventExecutor in the DesktopEnvironment destructor; is this pre-emptive
preparation for the multi-process implementation?

[alexeypa (please no reviews), 2012/08/31 23:13:30]

There is no real reason to keep it here at the moment. I removed it.

 
  private:
-  DesktopEnvironment(ChromotingHostContext* context,
-                     scoped_ptr<VideoFrameCapturer> capturer,
-                     scoped_ptr<EventExecutor> event_executor,
-                     scoped_ptr<AudioCapturer> audio_capturer);
-
-  // Host context used to make sure operations are run on the correct thread.
-  // This is owned by the ChromotingHost.
-  ChromotingHostContext* context_;
-
-  // Used to capture video to deliver to clients.
-  scoped_ptr<VideoFrameCapturer> capturer_;
-
   // Used to capture audio to deliver to clients.
   scoped_ptr<AudioCapturer> audio_capturer_;
 
   // Executes input and clipboard events received from the client.
   scoped_ptr<EventExecutor> event_executor_;
 
+  // Used to capture video to deliver to clients.
+  scoped_ptr<VideoFrameCapturer> video_capturer_;
+
   DISALLOW_COPY_AND_ASSIGN(DesktopEnvironment);
 };
 
 }  // namespace remoting
 
 #endif  // REMOTING_HOST_DESKTOP_ENVIRONMENT_H_