Support for two audio codec lists down into WebRtcVoiceEngine.

webrtc/pc/mediasession.cc

 /*
  *  Copyright 2004 The WebRTC project authors. All Rights Reserved.
  *
  *  Use of this source code is governed by a BSD-style license
  *  that can be found in the LICENSE file in the root of the source
  *  tree. An additional intellectual property rights grant can be found
  *  in the file PATENTS.  All contributing project authors may
  *  be found in the AUTHORS file in the root of the source tree.
  */
 
@@ skipping 32 matching lines @@
   func(&crypto_suites);
   for (const auto crypto : crypto_suites) {
     names->push_back(rtc::SrtpCryptoSuiteToName(crypto));
   }
 #endif
 }
 }  // namespace
 
 namespace cricket {
 
+namespace {
+std::vector<AudioCodec> MergeAudioCodecLists(
+    const std::vector<AudioCodec>& send_codecs,
+    const std::vector<AudioCodec>& recv_codecs);
+}
 
 // RTP Profile names
 // http://www.iana.org/assignments/rtp-parameters/rtp-parameters.xml
 // RFC4585
 const char kMediaProtocolAvpf[] = "RTP/AVPF";
 // RFC5124
 const char kMediaProtocolDtlsSavpf[] = "UDP/TLS/RTP/SAVPF";
 
 // We always generate offers with "UDP/TLS/RTP/SAVPF" when using DTLS-SRTP,
 // but we tolerate "RTP/SAVPF" in offers we receive, for compatibility.
@@ skipping 1182 matching lines @@
       add_legacy_(true),
       transport_desc_factory_(transport_desc_factory) {
 }
 
 MediaSessionDescriptionFactory::MediaSessionDescriptionFactory(
     ChannelManager* channel_manager,
     const TransportDescriptionFactory* transport_desc_factory)
     : secure_(SEC_DISABLED),
       add_legacy_(true),
       transport_desc_factory_(transport_desc_factory) {
-  channel_manager->GetSupportedAudioCodecs(&audio_sendrecv_codecs_);
+  channel_manager->GetSupportedAudioSendCodecs(&audio_send_codecs_);
+  channel_manager->GetSupportedAudioReceiveCodecs(&audio_recv_codecs_);
+  channel_manager->GetSupportedAudioSendCodecs(&audio_send_codecs_);
   channel_manager->GetSupportedAudioRtpHeaderExtensions(&audio_rtp_extensions_);
   channel_manager->GetSupportedVideoCodecs(&video_codecs_);
   channel_manager->GetSupportedVideoRtpHeaderExtensions(&video_rtp_extensions_);
   channel_manager->GetSupportedDataCodecs(&data_codecs_);
-  audio_send_codecs_ = audio_sendrecv_codecs_;
-  audio_recv_codecs_ = audio_sendrecv_codecs_;
+  audio_sendrecv_codecs_ =
+      MergeAudioCodecLists(audio_send_codecs_, audio_recv_codecs_);
 }
 
 const AudioCodecs& MediaSessionDescriptionFactory::audio_codecs() const {

[pthatcher1, 2016/05/31 18:26:48]

I don't think this is called by anything except one test in
mediasession_unittest.cc.  A better name for it would be audio_sendrev_codecs().
 Can you rename it that?

[ossu, 2016/06/01 08:34:12]

Good call! Will do that.

   return audio_sendrecv_codecs_;
 }
 
 const AudioCodecs& MediaSessionDescriptionFactory::audio_send_codecs() const {
   return audio_send_codecs_;
 }
 
 const AudioCodecs& MediaSessionDescriptionFactory::audio_recv_codecs() const {
   return audio_recv_codecs_;
 }
 
 void MediaSessionDescriptionFactory::set_audio_codecs(
     const AudioCodecs& send_and_recv_codecs) {
   audio_send_codecs_ = send_and_recv_codecs;
   audio_recv_codecs_ = send_and_recv_codecs;
   audio_sendrecv_codecs_ = send_and_recv_codecs;
 }
 
 void MediaSessionDescriptionFactory::set_audio_codecs(
     const AudioCodecs& send_codecs, const AudioCodecs& recv_codecs) {
   audio_send_codecs_ = send_codecs;
   audio_recv_codecs_ = recv_codecs;
-  audio_sendrecv_codecs_.clear();
-
-  // Intersect the two lists of codecs, preserving the order of the send codecs.
-  // If there's any difference in priorities, chances are encoding is more
-  // expensive than decoding, and high-priority send codecs are likely handled
-  // better (e.g. through hardware encoder support) than low-priority ones.
-  // TODO(ossu): This is O(n^2). However, each list should generally be small
-  // enough for this to not be a problem. They are also nicely local in memory
-  // like this.
-  for (const auto& sc : audio_send_codecs_) {
-    for (const auto& rc : audio_recv_codecs_) {
-      const size_t send_channels = (sc.channels == 0) ? 1 : sc.channels;
-      const size_t recv_channels = (rc.channels == 0) ? 1 : rc.channels;
-      if (sc.clockrate == rc.clockrate &&
-          sc.bitrate == rc.bitrate &&
-          send_channels == recv_channels &&
-          (_stricmp(sc.name.c_str(), rc.name.c_str()) == 0) &&
-          sc.params == rc.params &&
-          sc.feedback_params == rc.feedback_params) {
-        AudioCodec out_codec = sc;
-        out_codec.channels = send_channels;
-        audio_sendrecv_codecs_.push_back(out_codec);
-        break;
-      }
-    }
-  }
+  audio_sendrecv_codecs_ =
+      MergeAudioCodecLists(audio_send_codecs_, audio_recv_codecs_);
 }
 
 SessionDescription* MediaSessionDescriptionFactory::CreateOffer(
     const MediaSessionOptions& options,
     const SessionDescription* current_description) const {
   std::unique_ptr<SessionDescription> offer(new SessionDescription());
 
   StreamParamsVec current_streams;
   GetCurrentStreamParams(current_description, &current_streams);
 
@@ skipping 785 matching lines @@
   return static_cast<const VideoContentDescription*>(
       GetFirstMediaContentDescription(sdesc, MEDIA_TYPE_VIDEO));
 }
 
 const DataContentDescription* GetFirstDataContentDescription(
     const SessionDescription* sdesc) {
   return static_cast<const DataContentDescription*>(
       GetFirstMediaContentDescription(sdesc, MEDIA_TYPE_DATA));
 }
 
+namespace {
+std::vector<AudioCodec> MergeAudioCodecLists(

[pthatcher1, 2016/05/31 18:26:48]

Can you call this MergeSendRecvCodecs?

[ossu, 2016/06/01 08:34:12]

Sure thing!

+    const std::vector<AudioCodec>& send_codecs,
+    const std::vector<AudioCodec>& recv_codecs) {
+  std::vector<AudioCodec> merged;
+  // Intersect the two lists of codecs, preserving the order of the send codecs.
+  // If there's any difference in priorities, chances are encoding is more
+  // expensive than decoding, and high-priority send codecs are likely handled
+  // better (e.g. through hardware encoder support) than low-priority ones.
+  // TODO(ossu): This is O(n^2). However, each list should generally be small
+  // enough for this to not be a problem. They are also nicely local in memory
+  // like this.
+  for (const auto& sc : send_codecs) {
+    for (const auto& rc : recv_codecs) {
+      const size_t send_channels = (sc.channels == 0) ? 1 : sc.channels;
+      const size_t recv_channels = (rc.channels == 0) ? 1 : rc.channels;
+      if (sc.clockrate == rc.clockrate && sc.bitrate == rc.bitrate &&

[ossu, 2016/05/26 14:18:50]

This got clang-formatted. I'll put it back to one comparison per line, to make
it easier to read.

+          send_channels == recv_channels &&
+          (_stricmp(sc.name.c_str(), rc.name.c_str()) == 0) &&
+          sc.params == rc.params && sc.feedback_params == rc.feedback_params) {
+        AudioCodec out_codec = sc;
+        out_codec.channels = send_channels;
+        merged.push_back(out_codec);
+        break;
+      }
+    }
+  }
+
+  return merged;
+}
+}
+
 }  // namespace cricket