From 4261a73b56ada3608e8bec550ee60697d34fd614 Mon Sep 17 00:00:00 2001 From: Mirko Bonadei Date: Mon, 19 Jul 2021 14:28:48 +0200 Subject: [PATCH] Move SetVideoCodecs closer to AddVideoConfig. Bug: b/192821182 Change-Id: I8ab604abf780cd271d0890268da5ef5880677d55 Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/226460 Commit-Queue: Mirko Bonadei Reviewed-by: Artem Titov Cr-Commit-Position: refs/heads/master@{#34503} --- api/test/peerconnection_quality_test_fixture.h | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/api/test/peerconnection_quality_test_fixture.h b/api/test/peerconnection_quality_test_fixture.h index 1619bd7169..ea230f0ad0 100644 --- a/api/test/peerconnection_quality_test_fixture.h +++ b/api/test/peerconnection_quality_test_fixture.h @@ -349,6 +349,14 @@ class PeerConnectionE2EQualityTestFixture { virtual PeerConfigurer* AddVideoConfig( VideoConfig config, CapturingDeviceIndex capturing_device_index) = 0; + // Set the list of video codecs used by the peer during the test. These + // codecs will be negotiated in SDP during offer/answer exchange. The order + // of these codecs during negotiation will be the same as in |video_codecs|. + // Codecs have to be available in codecs list provided by peer connection to + // be negotiated. If some of specified codecs won't be found, the test will + // crash. + virtual PeerConfigurer* SetVideoCodecs( + std::vector video_codecs) = 0; // Set the audio stream for the call from this peer. If this method won't // be invoked, this peer will send no audio. virtual PeerConfigurer* SetAudioConfig(AudioConfig config) = 0; @@ -364,14 +372,6 @@ class PeerConnectionE2EQualityTestFixture { // applied to all summed RTP streams for this peer. virtual PeerConfigurer* SetBitrateSettings( BitrateSettings bitrate_settings) = 0; - // Set the list of video codecs used by the peer during the test. These - // codecs will be negotiated in SDP during offer/answer exchange. The order - // of these codecs during negotiation will be the same as in |video_codecs|. - // Codecs have to be available in codecs list provided by peer connection to - // be negotiated. If some of specified codecs won't be found, the test will - // crash. - virtual PeerConfigurer* SetVideoCodecs( - std::vector video_codecs) = 0; }; // Contains configuration for echo emulator.