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

Unified Diff: media/ffmpeg/ffmpeg_common_unittest.cc

Issue 2435603009: Use ffmpeg for opus decoding, no need to maintain our decoder. (Closed)
Patch Set: Created 4 years, 2 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/ffmpeg/ffmpeg_common_unittest.cc
diff --git a/media/ffmpeg/ffmpeg_common_unittest.cc b/media/ffmpeg/ffmpeg_common_unittest.cc
index 098417ad9a07f70e4ede34cbb9ea542f56952013..9aac8964b792af92f9d67184f29d1a3cf5e123d8 100644
--- a/media/ffmpeg/ffmpeg_common_unittest.cc
+++ b/media/ffmpeg/ffmpeg_common_unittest.cc
@@ -122,23 +122,6 @@ TEST_F(FFmpegCommonTest, AVStreamToDecoderConfig) {
ASSERT_TRUE(found_video);
}
-TEST_F(FFmpegCommonTest, OpusAudioDecoderConfig) {
- AVCodecContext context = {0};
- context.codec_type = AVMEDIA_TYPE_AUDIO;
- context.codec_id = AV_CODEC_ID_OPUS;
- context.channel_layout = CHANNEL_LAYOUT_STEREO;
- context.channels = 2;
- context.sample_fmt = AV_SAMPLE_FMT_FLT;
-
- // During conversion this sample rate should be changed to 48kHz.
- context.sample_rate = 44100;
-
- AudioDecoderConfig decoder_config;
- ASSERT_TRUE(AVCodecContextToAudioDecoderConfig(&context, Unencrypted(),
- &decoder_config));
- EXPECT_EQ(48000, decoder_config.samples_per_second());
-}
-
TEST_F(FFmpegCommonTest, TimeBaseConversions) {
const int64_t test_data[][5] = {
{1, 2, 1, 500000, 1}, {1, 3, 1, 333333, 1}, {1, 3, 2, 666667, 2},

Powered by Google App Engine
This is Rietveld 408576698