Updated AudioDecoderFactory to list AudioCodecSpecs instead of SdpAudioFormats.

webrtc/modules/audio_coding/codecs/audio_format.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.
  */
 
@@ skipping 25 matching lines @@
   ~SdpAudioFormat();
 
   SdpAudioFormat& operator=(const SdpAudioFormat&);
   SdpAudioFormat& operator=(SdpAudioFormat&&);
   bool operator<(const SdpAudioFormat& b) const;
 
   std::string name;
   int clockrate_hz;
   int num_channels;
   Parameters parameters;
-  // Parameters feedback_parameters; ??
 };
 
 void swap(SdpAudioFormat& a, SdpAudioFormat& b);
 std::ostream& operator<<(std::ostream& os, const SdpAudioFormat& saf);
 
+struct AudioCodecSpec {
+  SdpAudioFormat format;
+  bool allow_comfort_noise;  // This encoder can be used with an external

[ivoc, 2016/07/08 13:06:07]

How about just putting this in the SdpAudioFormat class instead? It looks like
it would make the code a bit simpler. 

[ossu, 2016/07/08 13:17:12]

No, because it doesn't make sense in SdpAudioFormat. It's information about how
the decoder (and eventually encoder) can be used by the audio coding module, not
about the audio format itself. It doesn't channel any information to/from SDP
nor is it useful information when creating the decoder/encoder.

[ivoc, 2016/07/08 13:37:34]

Acknowledged.

+                             // comfort noise generator.
+};
+
 }  // namespace webrtc
 
 #endif  // WEBRTC_MODULES_AUDIO_CODING_CODECS_AUDIO_FORMAT_H_