Pass SdpAudioFormat through Channel, without converting to CodecInst

webrtc/media/engine/payload_type_mapper.h

 /*
  *  Copyright (c) 2016 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.
  */
 
 #ifndef WEBRTC_MEDIA_ENGINE_PAYLOAD_TYPE_MAPPER_H_
 #define WEBRTC_MEDIA_ENGINE_PAYLOAD_TYPE_MAPPER_H_
 
 #include <map>
 #include <set>
 
 #include "webrtc/base/optional.h"
 #include "webrtc/media/base/codec.h"
 #include "webrtc/modules/audio_coding/codecs/audio_format.h"
 
 namespace cricket {
 
+webrtc::SdpAudioFormat AudioCodecToSdp(const AudioCodec& ac);

[the sun, 2016/12/08 10:53:52]

nit: "Sdp" -> "SdpAudioFormat", or maybe just call it FromAudioCodec()?

[kwiberg-webrtc, 2016/12/09 02:39:01]

Done.

+
 class PayloadTypeMapper {
  public:
   PayloadTypeMapper();
   ~PayloadTypeMapper();
 
   // Finds the current payload type for |format| or assigns a new one, if no
   // current mapping exists. Will return an empty value if it was unable to
   // create a mapping, i.e. if all dynamic payload type ids have been used up.
   rtc::Optional<int> GetMappingFor(const webrtc::SdpAudioFormat& format);
 
@@ skipping 12 matching lines @@
   };
 
   int next_unused_payload_type_;
   int max_payload_type_;
   std::map<webrtc::SdpAudioFormat, int, SdpAudioFormatOrdering> mappings_;
   std::set<int> used_payload_types_;
 };
 
 }  // namespace cricket
 #endif  // WEBRTC_MEDIA_ENGINE_PAYLOAD_TYPE_MAPPER_H_