Expose new video codec factories in the PeerConnectionFactory API

webrtc/api/peerconnectioninterface.h

Index: webrtc/api/peerconnectioninterface.h
diff --git a/webrtc/api/peerconnectioninterface.h b/webrtc/api/peerconnectioninterface.h
index 6a2b904453f9afd89917a7ab93f60a84b6c12902..a1c2b89807e5e8344a81906a567c589375bcad8b 100644
--- a/webrtc/api/peerconnectioninterface.h
+++ b/webrtc/api/peerconnectioninterface.h
@@ -112,6 +112,8 @@ class AudioDeviceModule;
 class AudioMixer;
 class CallFactoryInterface;
 class MediaConstraintsInterface;
+class VideoDecoderFactory;
+class VideoEncoderFactory;
 
 // MediaStream container interface.
 class StreamCollectionInterface : public rtc::RefCountInterface {
@@ -1113,6 +1115,21 @@ rtc::scoped_refptr<PeerConnectionFactoryInterface> CreatePeerConnectionFactory(
     rtc::scoped_refptr<AudioMixer> audio_mixer,
     rtc::scoped_refptr<AudioProcessing> audio_processing);
 
+// Create a new instance of PeerConnectionFactoryInterface with optional video
+// codec factories. These video factories represents all video codecs, i.e. no
+// extra internal video codecs will be added.
+rtc::scoped_refptr<PeerConnectionFactoryInterface> CreatePeerConnectionFactory(
+    rtc::Thread* network_thread,
+    rtc::Thread* worker_thread,
+    rtc::Thread* signaling_thread,
+    rtc::scoped_refptr<AudioDeviceModule> default_adm,
+    rtc::scoped_refptr<AudioEncoderFactory> audio_encoder_factory,
+    rtc::scoped_refptr<AudioDecoderFactory> audio_decoder_factory,
+    std::unique_ptr<VideoEncoderFactory> video_encoder_factory,
+    std::unique_ptr<VideoDecoderFactory> video_decoder_factory,
+    rtc::scoped_refptr<AudioMixer> audio_mixer,
+    rtc::scoped_refptr<AudioProcessing> audio_processing);

[Taylor Brandstetter, 2017/09/15 00:42:51]

nit: Maybe it's time to make a struct that contains all these arguments? See
https://bugs.chromium.org/p/webrtc/issues/detail?id=7913

[magjed_webrtc, 2017/09/15 13:39:16]

I totally agree, these create functions are obviously telescoping and we need
some kind of builder pattern. I'm not sure I'm the right person to fix it though
since I'm not an owner of this part of the code base, and I can imagine people
have a lot of opinions how this should be done. I'll go ahead and land this
first regardless.

+
 // Create a new instance of PeerConnectionFactoryInterface with external audio
 // mixer.
 //