Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(206)

Unified Diff: remoting/host/desktop_environment_factory.cc

Issue 11018004: Fix ChromotingHost and DesktopEnvironmentFactory references to TaskRunners. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: remoting/host/desktop_environment_factory.cc
diff --git a/remoting/host/desktop_environment_factory.cc b/remoting/host/desktop_environment_factory.cc
index 46d6dabb8e218487d31abae92f9a42c1ef110bd4..00ea4de7280bbbd9eab0a4eb95b7cf5e71442441 100644
--- a/remoting/host/desktop_environment_factory.cc
+++ b/remoting/host/desktop_environment_factory.cc
@@ -12,23 +12,22 @@
namespace remoting {
-DesktopEnvironmentFactory::DesktopEnvironmentFactory() {
+DesktopEnvironmentFactory::DesktopEnvironmentFactory(
+ scoped_refptr<base::SingleThreadTaskRunner> input_task_runner,
+ scoped_refptr<base::SingleThreadTaskRunner> ui_task_runner)
+ : input_task_runner_(input_task_runner), ui_task_runner_(ui_task_runner) {
alexeypa (please no reviews) 2012/10/01 18:31:34 nit: I believe it will look better if you put inpu
Wez 2012/10/02 05:19:44 Done.
}
DesktopEnvironmentFactory::~DesktopEnvironmentFactory() {
}
-scoped_ptr<DesktopEnvironment> DesktopEnvironmentFactory::Create(
- ChromotingHostContext* context) {
+scoped_ptr<DesktopEnvironment> DesktopEnvironmentFactory::Create() {
scoped_ptr<AudioCapturer> audio_capturer = AudioCapturer::Create();
scoped_ptr<EventExecutor> event_executor = EventExecutor::Create(
- context->desktop_task_runner(),
- context->ui_task_runner());
+ input_task_runner_, ui_task_runner_);
scoped_ptr<VideoFrameCapturer> video_capturer(VideoFrameCapturer::Create());
return scoped_ptr<DesktopEnvironment>(new DesktopEnvironment(
- audio_capturer.Pass(),
- event_executor.Pass(),
- video_capturer.Pass()));
+ audio_capturer.Pass(), event_executor.Pass(), video_capturer.Pass()));
}
bool DesktopEnvironmentFactory::SupportsAudioCapture() const {

Powered by Google App Engine
This is Rietveld 408576698