diff --git a/pc/webrtc_sdp.cc b/pc/webrtc_sdp.cc index c942689b8f..ae4e58d9f4 100644 --- a/pc/webrtc_sdp.cc +++ b/pc/webrtc_sdp.cc @@ -3636,9 +3636,9 @@ bool ParseRtpmapAttribute(absl::string_view line, AudioContentDescription* audio_desc = media_desc->as_audio(); for (const cricket::AudioCodec& existing_codec : audio_desc->codecs()) { + // TODO(crbug.com/1338902) re-add check for clockrate. if (!existing_codec.name.empty() && payload_type == existing_codec.id && (!absl::EqualsIgnoreCase(encoding_name, existing_codec.name) || - clock_rate != existing_codec.clockrate || channels != existing_codec.channels)) { rtc::StringBuilder description; description diff --git a/pc/webrtc_sdp_unittest.cc b/pc/webrtc_sdp_unittest.cc index 02f990b381..7880af0693 100644 --- a/pc/webrtc_sdp_unittest.cc +++ b/pc/webrtc_sdp_unittest.cc @@ -4665,11 +4665,10 @@ TEST_F(WebRtcSdpTest, DuplicateAudioRtpmapWithConflict) { "s=-\r\n" "t=0 0\r\n" "m=audio 49232 RTP/AVP 108\r\n" - // Same name but different payload type. "a=rtpmap:108 ISAC/16000\r\n" - "a=rtpmap:108 ISAC/32000\r\n"; + "a=rtpmap:108 G711/16000\r\n"; - ExpectParseFailure(sdp, "a=rtpmap:108 ISAC/32000"); + ExpectParseFailure(sdp, "a=rtpmap:108 G711/16000"); } TEST_F(WebRtcSdpTest, DuplicateVideoRtpmapWithConflict) {