Index: media/filters/ffmpeg_video_decoder_unittest.cc |
diff --git a/media/filters/ffmpeg_video_decoder_unittest.cc b/media/filters/ffmpeg_video_decoder_unittest.cc |
index 160d18f24a5bd25125c4e9371417b693448baa1f..e902bd04f1d77a0ef8b88806051486f7f20a751c 100644 |
--- a/media/filters/ffmpeg_video_decoder_unittest.cc |
+++ b/media/filters/ffmpeg_video_decoder_unittest.cc |
@@ -68,16 +68,16 @@ |
InitializeWithConfig(TestVideoConfig::Normal()); |
} |
- void InitializeWithConfigWithResult(const VideoDecoderConfig& config, |
- bool success) { |
- decoder_->Initialize(config, false, NewExpectedBoolCB(success), |
+ void InitializeWithConfigAndStatus(const VideoDecoderConfig& config, |
+ PipelineStatus status) { |
+ decoder_->Initialize(config, false, NewExpectedStatusCB(status), |
base::Bind(&FFmpegVideoDecoderTest::FrameReady, |
base::Unretained(this))); |
message_loop_.RunUntilIdle(); |
} |
void InitializeWithConfig(const VideoDecoderConfig& config) { |
- InitializeWithConfigWithResult(config, true); |
+ InitializeWithConfigAndStatus(config, PIPELINE_OK); |
} |
void Reinitialize() { |
@@ -217,7 +217,8 @@ |
TEST_F(FFmpegVideoDecoderTest, Initialize_UnsupportedDecoder) { |
// Test avcodec_find_decoder() returning NULL. |
- InitializeWithConfigWithResult(TestVideoConfig::Invalid(), false); |
+ InitializeWithConfigAndStatus(TestVideoConfig::Invalid(), |
+ DECODER_ERROR_NOT_SUPPORTED); |
} |
TEST_F(FFmpegVideoDecoderTest, Initialize_UnsupportedPixelFormat) { |
@@ -226,7 +227,7 @@ |
VideoFrame::UNKNOWN, |
kCodedSize, kVisibleRect, kNaturalSize, |
NULL, 0, false); |
- InitializeWithConfigWithResult(config, false); |
+ InitializeWithConfigAndStatus(config, DECODER_ERROR_NOT_SUPPORTED); |
} |
TEST_F(FFmpegVideoDecoderTest, Initialize_OpenDecoderFails) { |
@@ -235,7 +236,7 @@ |
kVideoFormat, |
kCodedSize, kVisibleRect, kNaturalSize, |
NULL, 0, false); |
- InitializeWithConfigWithResult(config, false); |
+ InitializeWithConfigAndStatus(config, DECODER_ERROR_NOT_SUPPORTED); |
} |
TEST_F(FFmpegVideoDecoderTest, Initialize_AspectRatioNumeratorZero) { |
@@ -249,7 +250,7 @@ |
NULL, |
0, |
false); |
- InitializeWithConfigWithResult(config, false); |
+ InitializeWithConfigAndStatus(config, DECODER_ERROR_NOT_SUPPORTED); |
} |
TEST_F(FFmpegVideoDecoderTest, Initialize_AspectRatioDenominatorZero) { |
@@ -263,7 +264,7 @@ |
NULL, |
0, |
false); |
- InitializeWithConfigWithResult(config, false); |
+ InitializeWithConfigAndStatus(config, DECODER_ERROR_NOT_SUPPORTED); |
} |
TEST_F(FFmpegVideoDecoderTest, Initialize_AspectRatioNumeratorNegative) { |
@@ -277,7 +278,7 @@ |
NULL, |
0, |
false); |
- InitializeWithConfigWithResult(config, false); |
+ InitializeWithConfigAndStatus(config, DECODER_ERROR_NOT_SUPPORTED); |
} |
TEST_F(FFmpegVideoDecoderTest, Initialize_AspectRatioDenominatorNegative) { |
@@ -291,7 +292,7 @@ |
NULL, |
0, |
false); |
- InitializeWithConfigWithResult(config, false); |
+ InitializeWithConfigAndStatus(config, DECODER_ERROR_NOT_SUPPORTED); |
} |
TEST_F(FFmpegVideoDecoderTest, Initialize_AspectRatioNumeratorTooLarge) { |
@@ -307,7 +308,7 @@ |
NULL, |
0, |
false); |
- InitializeWithConfigWithResult(config, false); |
+ InitializeWithConfigAndStatus(config, DECODER_ERROR_NOT_SUPPORTED); |
} |
TEST_F(FFmpegVideoDecoderTest, Initialize_AspectRatioDenominatorTooLarge) { |
@@ -322,7 +323,7 @@ |
NULL, |
0, |
false); |
- InitializeWithConfigWithResult(config, false); |
+ InitializeWithConfigAndStatus(config, DECODER_ERROR_NOT_SUPPORTED); |
} |
TEST_F(FFmpegVideoDecoderTest, Reinitialize_Normal) { |
@@ -332,7 +333,8 @@ |
TEST_F(FFmpegVideoDecoderTest, Reinitialize_Failure) { |
Initialize(); |
- InitializeWithConfigWithResult(TestVideoConfig::Invalid(), false); |
+ InitializeWithConfigAndStatus(TestVideoConfig::Invalid(), |
+ DECODER_ERROR_NOT_SUPPORTED); |
} |
TEST_F(FFmpegVideoDecoderTest, Reinitialize_AfterDecodeFrame) { |