WebRtcVoiceMediaChannel::AddRecvStream: Don't call SetRecPayloadType

webrtc/voice_engine/channel.cc

Index: webrtc/voice_engine/channel.cc
diff --git a/webrtc/voice_engine/channel.cc b/webrtc/voice_engine/channel.cc
index 1d73db6d396a29ab1bc3ca12beee0c8ce1106ae0..0071a1eca61effd12dedf240c3fba87b14248fd9 100644
--- a/webrtc/voice_engine/channel.cc
+++ b/webrtc/voice_engine/channel.cc
@@ -1323,6 +1323,14 @@ int32_t Channel::GetVADStatus(bool& enabledVAD,
   return 0;
 }
 
+bool Channel::SetReceiveCodecs(const std::map<int, SdpAudioFormat>& codecs) {
+  if (!rtp_payload_registry_->SetAudioReceivePayloads(codecs)) {

[the sun, 2017/03/14 20:07:01]

If you remove the return value from the above call, we don't need it in
Channel::SetReceiveCodecs(), or the ChannelProxy either.

[kwiberg-webrtc, 2017/03/16 10:24:12]

Acknowledged. I'll remove as many of them as I can.

+    return false;
+  }
+  audio_coding_->SetReceiveCodecs(codecs);
+  return true;
+}
+
 int32_t Channel::SetRecPayloadType(const CodecInst& codec) {
   return SetRecPayloadType(codec.pltype, CodecInstToSdp(codec));
 }