From 1126a186f6fd422b2872e5fb38a6dfe826e522d9 Mon Sep 17 00:00:00 2001 From: Philipp Hancke Date: Mon, 6 Jul 2020 22:30:16 +0200 Subject: [PATCH] red: add red closer to opus in the SDP this makes the association between opus and red a bit more obvious. Also it allows access to the opus payload type which might be used in the fmtp line in a future CL BUG=webrtc:11640 Change-Id: I04e0648aedf049d103e3c3481c8712dfc9b79538 Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/178800 Commit-Queue: Harald Alvestrand Reviewed-by: Karl Wiberg Reviewed-by: Harald Alvestrand Cr-Commit-Position: refs/heads/master@{#31868} --- media/engine/webrtc_voice_engine.cc | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/media/engine/webrtc_voice_engine.cc b/media/engine/webrtc_voice_engine.cc index 590c31e409..749f2506a5 100644 --- a/media/engine/webrtc_voice_engine.cc +++ b/media/engine/webrtc_voice_engine.cc @@ -722,6 +722,11 @@ std::vector WebRtcVoiceEngine::CollectCodecs( } out.push_back(codec); + + if (codec.name == kOpusCodecName && + IsAudioRedForOpusFieldTrialEnabled()) { + map_format({kRedCodecName, 48000, 2}, &out); + } } } @@ -732,11 +737,6 @@ std::vector WebRtcVoiceEngine::CollectCodecs( } } - // Add red codec. - if (IsAudioRedForOpusFieldTrialEnabled()) { - map_format({kRedCodecName, 48000, 2}, &out); - } - // Add telephone-event codecs last. for (const auto& dtmf : generate_dtmf) { if (dtmf.second) {