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

Unified Diff: media/filters/ffmpeg_audio_decoder_unittest.cc

Issue 9632024: Create video and audio decoder threads on demand. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Switched to base::Bind() and reverted gyp change+new files 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/filters/ffmpeg_audio_decoder_unittest.cc
diff --git a/media/filters/ffmpeg_audio_decoder_unittest.cc b/media/filters/ffmpeg_audio_decoder_unittest.cc
index eb634c7926b6f7f5d2e4cf33bc3436473604c26c..03961244b914ac78c9bbaac3eeb503cdb61f22b5 100644
--- a/media/filters/ffmpeg_audio_decoder_unittest.cc
+++ b/media/filters/ffmpeg_audio_decoder_unittest.cc
@@ -20,16 +20,22 @@ using ::testing::_;
using ::testing::ReturnRef;
using ::testing::StrictMock;
+namespace {
Ami GONE FROM CHROMIUM 2012/03/09 16:15:06 s/anon namespace/static/ in media code
tommi (sloooow) - chröme 2012/03/11 18:49:36 Done.
+// Used to inject our message loop into the FFmpegAudioDecoder.
+template<class T> T Identity(T t) { return t; }
+} // end namespace
+
namespace media {
ACTION_P(InvokeReadPacket, test) {
test->ReadPacket(arg0);
}
-
+
class FFmpegAudioDecoderTest : public testing::Test {
- public:
+ public:
FFmpegAudioDecoderTest()
- : decoder_(new FFmpegAudioDecoder(&message_loop_)),
+ : decoder_(new FFmpegAudioDecoder(base::Bind(&Identity<MessageLoop*>,
+ &message_loop_))),
demuxer_(new StrictMock<MockDemuxerStream>()) {
CHECK(FFmpegGlue::GetInstance());

Powered by Google App Engine
This is Rietveld 408576698