Injectable audio encoders: BuiltinAudioEncoderFactory

webrtc/modules/audio_coding/codecs/g711/audio_encoder_pcm.h

Index: webrtc/modules/audio_coding/codecs/g711/audio_encoder_pcm.h
diff --git a/webrtc/modules/audio_coding/codecs/g711/audio_encoder_pcm.h b/webrtc/modules/audio_coding/codecs/g711/audio_encoder_pcm.h
index 721344528f8b2e7bb801cbf2cdc2df2faceadaba..ab60eb097f64cd96a7c9f728a59875c20c573e79 100644
--- a/webrtc/modules/audio_coding/codecs/g711/audio_encoder_pcm.h
+++ b/webrtc/modules/audio_coding/codecs/g711/audio_encoder_pcm.h
@@ -13,6 +13,7 @@
 
 #include <vector>
 
+#include "webrtc/api/audio_codecs/audio_format.h"
 #include "webrtc/base/constructormagic.h"
 #include "webrtc/modules/audio_coding/codecs/audio_encoder.h"
 
@@ -80,6 +81,11 @@ class AudioEncoderPcmA final : public AudioEncoderPcm {
   explicit AudioEncoderPcmA(const Config& config)
       : AudioEncoderPcm(config, kSampleRateHz) {}
   explicit AudioEncoderPcmA(const CodecInst& codec_inst);
+  explicit AudioEncoderPcmA(int payload_type, const SdpAudioFormat& format);

[kwiberg-webrtc, 2017/02/19 21:41:10]

Why explicit?

[ossu, 2017/02/20 12:20:26]

Copied the line above, made changes, didn't think about it.

+
+  static constexpr const char* GetPayloadName() { return "pcma"; }
+  static rtc::Optional<AudioFormatInfo> QueryAudioFormat(
+      const SdpAudioFormat& format);
 
  protected:
   size_t EncodeCall(const int16_t* audio,
@@ -104,6 +110,11 @@ class AudioEncoderPcmU final : public AudioEncoderPcm {
   explicit AudioEncoderPcmU(const Config& config)
       : AudioEncoderPcm(config, kSampleRateHz) {}
   explicit AudioEncoderPcmU(const CodecInst& codec_inst);
+  explicit AudioEncoderPcmU(int payload_type, const SdpAudioFormat& format);
+
+  static constexpr const char* GetPayloadName() { return "pcmu"; }
+  static rtc::Optional<AudioFormatInfo> QueryAudioFormat(
+      const SdpAudioFormat& format);
 
  protected:
   size_t EncodeCall(const int16_t* audio,