Bug 1632489 - Enable repaired-rtp-stream-id extension in jsep; r=bwc
authorDan Minor <dminor@mozilla.com>
Fri, 15 May 2020 11:55:08 +0000
changeset 530291 58b080b54b6fe7b6ea081d279979655bb9de814d
parent 530290 4ca082adb91d5fdd84fe8d361dd064e42d2f4edc
child 530292 078662142da6e8acd5b8c0d06bda0934e01bdee6
push id37420
push usernerli@mozilla.com
push dateFri, 15 May 2020 21:52:36 +0000
treeherdermozilla-central@f340bbb582d1 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersbwc
bugs1632489
milestone78.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1632489 - Enable repaired-rtp-stream-id extension in jsep; r=bwc Differential Revision: https://phabricator.services.mozilla.com/D74839
media/webrtc/signaling/src/jsep/JsepSessionImpl.cpp
--- a/media/webrtc/signaling/src/jsep/JsepSessionImpl.cpp
+++ b/media/webrtc/signaling/src/jsep/JsepSessionImpl.cpp
@@ -401,16 +401,18 @@ std::vector<SdpExtmapAttributeList::Extm
     case SdpMediaSection::kVideo:
       mediaType = JsepMediaType::kVideo;
       if (msection.GetAttributeList().HasAttribute(
               SdpAttribute::kRidAttribute)) {
         // We need RID support
         // TODO: Would it be worth checking that the direction is sane?
         AddVideoRtpExtension(webrtc::RtpExtension::kRtpStreamIdUri,
                              SdpDirectionAttribute::kSendonly);
+        AddVideoRtpExtension(webrtc::RtpExtension::kRepairedRtpStreamIdUri,
+                             SdpDirectionAttribute::kSendonly);
       }
       break;
     default:;
   }
   if (mediaType != JsepMediaType::kNone) {
     for (auto ext = mRtpExtensions.begin(); ext != mRtpExtensions.end();
          ++ext) {
       if (ext->mMediaType == mediaType ||