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

Unified Diff: webrtc/audio/audio_send_stream_unittest.cc

Issue 2446963003: Clean up logging in AudioSendStream::SetupSendCodec(). (Closed)
Patch Set: fix build breakage? 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
« no previous file with comments | « webrtc/audio/audio_send_stream.cc ('k') | webrtc/config.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/audio/audio_send_stream_unittest.cc
diff --git a/webrtc/audio/audio_send_stream_unittest.cc b/webrtc/audio/audio_send_stream_unittest.cc
index 14ca0e076fe51889cfb92c0bba017e47b4e2e3aa..a2832ded00cb330d86ffae8805955326cb1033a2 100644
--- a/webrtc/audio/audio_send_stream_unittest.cc
+++ b/webrtc/audio/audio_send_stream_unittest.cc
@@ -219,12 +219,26 @@ TEST(AudioSendStreamTest, ConfigToString) {
RtpExtension(RtpExtension::kAbsSendTimeUri, kAbsSendTimeId));
config.rtp.c_name = kCName;
config.voe_channel_id = kChannelId;
+ config.min_bitrate_kbps = 12;
+ config.max_bitrate_kbps = 34;
+ config.send_codec_spec.nack_enabled = true;
+ config.send_codec_spec.transport_cc_enabled = false;
+ config.send_codec_spec.enable_codec_fec = true;
+ config.send_codec_spec.enable_opus_dtx = false;
+ config.send_codec_spec.opus_max_playback_rate = 32000;
config.send_codec_spec.cng_payload_type = 42;
+ config.send_codec_spec.cng_plfreq = 56;
+ config.send_codec_spec.codec_inst = kIsacCodec;
EXPECT_EQ(
"{rtp: {ssrc: 1234, extensions: [{uri: "
"http://www.webrtc.org/experiments/rtp-hdrext/abs-send-time, id: 3}], "
- "nack: {rtp_history_ms: 0}, c_name: foo_name}, voe_channel_id: 1, "
- "cng_payload_type: 42}",
+ "nack: {rtp_history_ms: 0}, c_name: foo_name}, send_transport: nullptr, "
+ "voe_channel_id: 1, min_bitrate_kbps: 12, max_bitrate_kbps: 34, "
+ "send_codec_spec: {nack_enabled: true, transport_cc_enabled: false, "
+ "enable_codec_fec: true, enable_opus_dtx: false, opus_max_playback_rate: "
+ "32000, cng_payload_type: 42, cng_plfreq: 56, codec_inst: {pltype: "
+ "103, plname: \"isac\", plfreq: 16000, pacsize: 320, channels: 1, rate: "
+ "32000}}}",
config.ToString());
}
« no previous file with comments | « webrtc/audio/audio_send_stream.cc ('k') | webrtc/config.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698