Bug 1363563: remove and erase existing header extensions. r?mjf
MozReview-Commit-ID: IzVEaOhLNwR
--- a/media/webrtc/signaling/src/media-conduit/VideoConduit.cpp
+++ b/media/webrtc/signaling/src/media-conduit/VideoConduit.cpp
@@ -249,19 +249,23 @@ WebrtcVideoConduit::~WebrtcVideoConduit(
}
void
WebrtcVideoConduit::AddLocalRTPExtensions(bool aIsSend,
const std::vector<webrtc::RtpExtension> & aExtensions)
{
auto& extList = aIsSend ? mSendStreamConfig.rtp.extensions :
mRecvStreamConfig.rtp.extensions;
- std::remove_if(extList.begin(), extList.end(), [&](const webrtc::RtpExtension & i) {
- return std::find(aExtensions.begin(), aExtensions.end(),i) != aExtensions.end();
- });
+ extList.erase(std::remove_if(extList.begin(),
+ extList.end(),
+ [&](const webrtc::RtpExtension & i) {
+ return std::find(aExtensions.begin(),
+ aExtensions.end(),
+ i) != aExtensions.end(); }),
+ extList.end());
extList.insert(extList.end(), aExtensions.begin(), aExtensions.end());
}
std::vector<webrtc::RtpExtension>
WebrtcVideoConduit::GetLocalRTPExtensions(bool aIsSend) const
{
return aIsSend ? mSendStreamConfig.rtp.extensions : mRecvStreamConfig.rtp.extensions;
}