Updated AudioDecoderFactory to list AudioCodecSpecs instead of SdpAudioFormats.

webrtc/modules/audio_coding/codecs/audio_format.h

Index: webrtc/modules/audio_coding/codecs/audio_format.h
diff --git a/webrtc/modules/audio_coding/codecs/audio_format.h b/webrtc/modules/audio_coding/codecs/audio_format.h
index 09075e49acba9b378af2aa47bc993cd9fc9181a2..6a9f315bdf7144e633d244aab751cd3e3fdc2cc9 100644
--- a/webrtc/modules/audio_coding/codecs/audio_format.h
+++ b/webrtc/modules/audio_coding/codecs/audio_format.h
@@ -43,12 +43,17 @@ struct SdpAudioFormat {
   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_