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

Unified Diff: webrtc/modules/audio_coding/neteq/audio_decoder_unittest.cc

Issue 2695243005: Injectable audio encoders: BuiltinAudioEncoderFactory (Closed)
Patch Set: Cleaned up parameter parsing in AudioCodecOpus Created 3 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: webrtc/modules/audio_coding/neteq/audio_decoder_unittest.cc
diff --git a/webrtc/modules/audio_coding/neteq/audio_decoder_unittest.cc b/webrtc/modules/audio_coding/neteq/audio_decoder_unittest.cc
index 1ee92a11c576d8f85c76d46428f37e8e8237030d..6bc3c0bffabaea68f14b49dc1f88f7ddec2ce7a7 100644
--- a/webrtc/modules/audio_coding/neteq/audio_decoder_unittest.cc
+++ b/webrtc/modules/audio_coding/neteq/audio_decoder_unittest.cc
@@ -611,8 +611,8 @@ void TestOpusSetTargetBitrates(AudioEncoder* audio_encoder) {
EXPECT_EQ(6000, SetAndGetTargetBitrate(audio_encoder, 5999));
EXPECT_EQ(6000, SetAndGetTargetBitrate(audio_encoder, 6000));
EXPECT_EQ(32000, SetAndGetTargetBitrate(audio_encoder, 32000));
- EXPECT_EQ(512000, SetAndGetTargetBitrate(audio_encoder, 512000));
- EXPECT_EQ(512000, SetAndGetTargetBitrate(audio_encoder, 513000));
+ EXPECT_EQ(510000, SetAndGetTargetBitrate(audio_encoder, 510000));
+ EXPECT_EQ(510000, SetAndGetTargetBitrate(audio_encoder, 510000));
kwiberg-webrtc 2017/03/15 13:33:19 The last of these four should be 511000, I believe
ossu 2017/03/16 18:03:58 Acknowledged.
}
} // namespace

Powered by Google App Engine
This is Rietveld 408576698