Add AudioEncoder::GetTargetBitrate

webrtc/modules/audio_coding/codecs/opus/audio_encoder_mutable_opus_test.cc

Index: webrtc/modules/audio_coding/codecs/opus/audio_encoder_mutable_opus_test.cc
diff --git a/webrtc/modules/audio_coding/codecs/opus/audio_encoder_mutable_opus_test.cc b/webrtc/modules/audio_coding/codecs/opus/audio_encoder_mutable_opus_test.cc
index ed3f52fff7f42052c2a1568a9746a60cffada84e..b585cb07b0182118d0043e474d24ef7c41c97974 100644
--- a/webrtc/modules/audio_coding/codecs/opus/audio_encoder_mutable_opus_test.cc
+++ b/webrtc/modules/audio_coding/codecs/opus/audio_encoder_mutable_opus_test.cc
@@ -79,6 +79,35 @@ TEST_F(AudioEncoderMutableOpusTest, ToggleDtx) {
   // Turn off DTX.
   EXPECT_TRUE(encoder_->SetDtx(false));
 }
+
+TEST_F(AudioEncoderMutableOpusTest, SetBitrate) {
+  CreateCodec(1);
+  // Constants are replicated from audio_encoder_opus.cc.
+  const int kMinBitrateBps = 500;
+  const int kMaxBitrateBps = 512000;
+  // Set a too low bitrate.
+  encoder_->SetTargetBitrate(kMinBitrateBps - 1);
+  EXPECT_EQ(kMinBitrateBps, encoder_->GetTargetBitrate());
+  EXPECT_EQ(kMinBitrateBps, encoder_->bitrate_bps());
+  // Set a too high bitrate.
+  encoder_->SetTargetBitrate(kMaxBitrateBps + 1);
+  EXPECT_EQ(kMaxBitrateBps, encoder_->GetTargetBitrate());
+  EXPECT_EQ(kMaxBitrateBps, encoder_->bitrate_bps());
+  // Set the minimum rate.
+  encoder_->SetTargetBitrate(kMinBitrateBps);
+  EXPECT_EQ(kMinBitrateBps, encoder_->GetTargetBitrate());
+  EXPECT_EQ(kMinBitrateBps, encoder_->bitrate_bps());
+  // Set the maximum rate.
+  encoder_->SetTargetBitrate(kMaxBitrateBps);
+  EXPECT_EQ(kMaxBitrateBps, encoder_->GetTargetBitrate());
+  EXPECT_EQ(kMaxBitrateBps, encoder_->bitrate_bps());
+  // Set rates from 1000 up to 32000 bps.
+  for (int rate = 1000; rate <= 32000; rate += 1000) {
+    encoder_->SetTargetBitrate(rate);
+    EXPECT_EQ(rate, encoder_->GetTargetBitrate());
+    EXPECT_EQ(rate, encoder_->bitrate_bps());
+  }

[kwiberg-webrtc, 2015/06/17 14:36:31]

This test is repetitive now, since GetTargetBitrate and bitrate_bps do exactly
the same thing.

[hlundin-webrtc, 2015/06/18 08:52:18]

Done.

+}
 #endif  // WEBRTC_CODEC_OPUS
 
 }  // namespace acm2