From 930ff23261b4cfd900a7c83ab7cbcff80d27fa6a Mon Sep 17 00:00:00 2001 From: minyue Date: Fri, 3 Mar 2017 07:34:52 -0800 Subject: [PATCH] Fixing a unittest to allow new Opus version. New Opus version starts to support 120ms frame lengths. AudioEncoderOpusTest had an unnecessary check on the available frame lengths. It is removed in this CL. BUG=b/35415318 Review-Url: https://codereview.webrtc.org/2731583003 Cr-Commit-Position: refs/heads/master@{#17011} --- .../audio_coding/codecs/opus/audio_encoder_opus_unittest.cc | 2 -- 1 file changed, 2 deletions(-) diff --git a/webrtc/modules/audio_coding/codecs/opus/audio_encoder_opus_unittest.cc b/webrtc/modules/audio_coding/codecs/opus/audio_encoder_opus_unittest.cc index 11456f2179..a6c502b598 100644 --- a/webrtc/modules/audio_coding/codecs/opus/audio_encoder_opus_unittest.cc +++ b/webrtc/modules/audio_coding/codecs/opus/audio_encoder_opus_unittest.cc @@ -274,8 +274,6 @@ TEST(AudioEncoderOpusTest, SetReceiverFrameLengthRange) { EXPECT_THAT(states.encoder->supported_frame_lengths_ms(), ElementsAre(states.encoder->next_frame_length_ms())); states.encoder->SetReceiverFrameLengthRange(0, 12345); - EXPECT_THAT(states.encoder->supported_frame_lengths_ms(), - ElementsAre(20, 60)); states.encoder->SetReceiverFrameLengthRange(21, 60); EXPECT_THAT(states.encoder->supported_frame_lengths_ms(), ElementsAre(60)); states.encoder->SetReceiverFrameLengthRange(20, 59);