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

Unified Diff: media/tools/player_wtl/movie.cc

Issue 9632024: Create video and audio decoder threads on demand. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Log an error when Initialize is called more than once. Created 8 years, 9 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: media/tools/player_wtl/movie.cc
diff --git a/media/tools/player_wtl/movie.cc b/media/tools/player_wtl/movie.cc
index b2e7acc4f74343c2c9fa69b6af7fabcefd67a5dc..3e862dcbf152523947d9d2ef4b5d74f45bea3c17 100644
--- a/media/tools/player_wtl/movie.cc
+++ b/media/tools/player_wtl/movie.cc
@@ -4,6 +4,7 @@
#include "media/tools/player_wtl/movie.h"
+#include "base/bind.h"
#include "base/memory/singleton.h"
#include "base/threading/platform_thread.h"
#include "base/utf_string_conversions.h"
@@ -80,9 +81,13 @@ bool Movie::Open(const wchar_t* url, VideoRendererBase* video_renderer) {
collection->SetDemuxerFactory(scoped_ptr<DemuxerFactory>(
new FFmpegDemuxerFactory(data_source, pipeline_loop)));
collection->AddAudioDecoder(new FFmpegAudioDecoder(
- message_loop_factory_->GetMessageLoop("AudioDecoderThread")));
+ base::Bind(&MessageLoopFactory::GetMessageLoop,
+ base::Unretained(message_loop_factory_.get()),
+ "AudioDecoderThread")));
collection->AddVideoDecoder(new FFmpegVideoDecoder(
- message_loop_factory_->GetMessageLoop("VideoDecoderThread")));
+ base::Bind(&MessageLoopFactory::GetMessageLoop,
+ base::Unretained(message_loop_factory_.get()),
+ "VideoDecoderThread")));
// TODO(vrk): Re-enabled audio. (crbug.com/112159)
collection->AddAudioRenderer(new media::NullAudioRenderer());

Powered by Google App Engine
This is Rietveld 408576698