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

Unified Diff: webrtc/pc/webrtcsdp_unittest.cc

Issue 2642923003: Removed double-special-casing of ISAC in libjingle and WebRtcVoE. (Closed)
Patch Set: Rebase Created 3 years, 11 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/pc/webrtcsdp.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/pc/webrtcsdp_unittest.cc
diff --git a/webrtc/pc/webrtcsdp_unittest.cc b/webrtc/pc/webrtcsdp_unittest.cc
index 54eb22e11a668777be7d5a1fda645a434212f502..43774abf7d72564c2940383fa06672942115b3ae 100644
--- a/webrtc/pc/webrtcsdp_unittest.cc
+++ b/webrtc/pc/webrtcsdp_unittest.cc
@@ -1128,8 +1128,8 @@ class WebRtcSdpTest : public testing::Test {
audio->set_protocol(cricket::kMediaProtocolSavpf);
AudioCodec opus(111, "opus", 48000, 0, 2);
audio->AddCodec(opus);
- audio->AddCodec(AudioCodec(103, "ISAC", 16000, 32000, 1));
- audio->AddCodec(AudioCodec(104, "ISAC", 32000, 56000, 1));
+ audio->AddCodec(AudioCodec(103, "ISAC", 16000, 0, 1));
+ audio->AddCodec(AudioCodec(104, "ISAC", 32000, 0, 1));
return audio;
}
@@ -1664,13 +1664,6 @@ class WebRtcSdpTest : public testing::Test {
cricket::AudioCodec codec = acd->codecs()[i];
VerifyCodecParameter(codec.params, "ptime", params.ptime);
VerifyCodecParameter(codec.params, "maxptime", params.max_ptime);
- if (codec.name == "ISAC") {
- if (codec.clockrate == 16000) {
- EXPECT_EQ(32000, codec.bitrate);
- } else {
- EXPECT_EQ(56000, codec.bitrate);
- }
- }
}
const ContentInfo* vc = GetFirstVideoContent(jdesc_output->description());
@@ -2275,7 +2268,7 @@ TEST_F(WebRtcSdpTest, DeserializeSessionDescriptionWithoutRtpmap) {
// the payload types (<fmt>s) on the m= line.
ref_codecs.push_back(AudioCodec(0, "PCMU", 8000, 0, 1));
ref_codecs.push_back(AudioCodec(18, "G729", 16000, 0, 1));
- ref_codecs.push_back(AudioCodec(103, "ISAC", 16000, 32000, 1));
+ ref_codecs.push_back(AudioCodec(103, "ISAC", 16000, 0, 1));
EXPECT_EQ(ref_codecs, audio->codecs());
}
« no previous file with comments | « webrtc/pc/webrtcsdp.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698