Chromium Code Reviews| Index: webrtc/media/engine/nullwebrtcvideoengine_unittest.cc |
| diff --git a/webrtc/media/engine/nullwebrtcvideoengine_unittest.cc b/webrtc/media/engine/nullwebrtcvideoengine_unittest.cc |
| index 31a2754e19a7a9eb4de357805f67a981556ffe3e..38d6e77488632fa32b50fd0e7ec715ce1d1f1f66 100644 |
| --- a/webrtc/media/engine/nullwebrtcvideoengine_unittest.cc |
| +++ b/webrtc/media/engine/nullwebrtcvideoengine_unittest.cc |
| @@ -18,19 +18,23 @@ namespace cricket { |
| class WebRtcMediaEngineNullVideo |
| : public CompositeMediaEngine<WebRtcVoiceEngine, NullWebRtcVideoEngine> { |
| public: |
| - WebRtcMediaEngineNullVideo(webrtc::AudioDeviceModule* adm, |
| - WebRtcVideoEncoderFactory* encoder_factory, |
| - WebRtcVideoDecoderFactory* decoder_factory) |
| - : CompositeMediaEngine<WebRtcVoiceEngine, NullWebRtcVideoEngine>(adm) { |
| - video_.SetExternalDecoderFactory(decoder_factory); |
| - video_.SetExternalEncoderFactory(encoder_factory); |
| + WebRtcMediaEngineNullVideo( |
| + webrtc::AudioDeviceModule* adm, |
| + const rtc::scoped_refptr<webrtc::AudioDecoderFactory>& |
| + audio_decoder_factory, |
| + WebRtcVideoEncoderFactory* video_encoder_factory, |
| + WebRtcVideoDecoderFactory* video_decoder_factory) |
| + : CompositeMediaEngine<WebRtcVoiceEngine, NullWebRtcVideoEngine>( |
| + adm, audio_decoder_factory) { |
| + video_.SetExternalDecoderFactory(video_decoder_factory); |
| + video_.SetExternalEncoderFactory(video_encoder_factory); |
| } |
| }; |
| // Simple test to check if NullWebRtcVideoEngine implements the methods |
| // required by CompositeMediaEngine. |
| TEST(NullWebRtcVideoEngineTest, CheckInterface) { |
| - WebRtcMediaEngineNullVideo engine(nullptr, nullptr, nullptr); |
| + WebRtcMediaEngineNullVideo engine(nullptr, nullptr, nullptr, nullptr); |
|
ossu
2016/05/19 16:08:47
This is just lovely! :)
|
| EXPECT_TRUE(engine.Init()); |
| } |