From d0e1de5361308913ca33c8c499bcd286345c39c3 Mon Sep 17 00:00:00 2001 From: "Byron Campen [:bwc]" Date: Thu, 14 May 2015 15:54:21 -0700 Subject: [PATCH] Bug 818640 - Test that using dynamic payload types < 96 works. r=mt --HG-- extra : transplant_source : %B0D%5D%EC%B2%3D%AC%A4%3B%0B%E1%7E%AC%10%96%AB%F9M%83%A1 --- .../signaling/test/jsep_session_unittest.cpp | 31 +++++++++++++++++++ 1 file changed, 31 insertions(+) diff --git a/media/webrtc/signaling/test/jsep_session_unittest.cpp b/media/webrtc/signaling/test/jsep_session_unittest.cpp index 5f504e9edbb8..ed044b3525a6 100644 --- a/media/webrtc/signaling/test/jsep_session_unittest.cpp +++ b/media/webrtc/signaling/test/jsep_session_unittest.cpp @@ -396,6 +396,18 @@ protected: return nullptr; } + void + SetPayloadTypeNumber(JsepSession& session, + const std::string& codecName, + const std::string& payloadType) + { + for (auto* codec : session.Codecs()) { + if (codec->mName == codecName) { + codec->mDefaultPt = payloadType; + } + } + } + void EnsureNegotiationFailure(SdpMediaSection::MediaType type, const std::string& codecName) @@ -3158,6 +3170,25 @@ TEST_F(JsepSessionTest, StronglyPreferredCodec) ASSERT_EQ("H264", codec->mName); } +TEST_F(JsepSessionTest, LowDynamicPayloadType) +{ + SetPayloadTypeNumber(mSessionOff, "opus", "12"); + types.push_back(SdpMediaSection::kAudio); + AddTracks(mSessionOff, "audio"); + AddTracks(mSessionAns, "audio"); + + OfferAnswer(); + const JsepCodecDescription* codec; + GetCodec(mSessionAns, 0, true, 0, &codec); // sending + ASSERT_TRUE(codec); + ASSERT_EQ("opus", codec->mName); + ASSERT_EQ("12", codec->mDefaultPt); + GetCodec(mSessionAns, 0, false, 0, &codec); // receiving + ASSERT_TRUE(codec); + ASSERT_EQ("opus", codec->mName); + ASSERT_EQ("12", codec->mDefaultPt); +} + } // namespace mozilla int