diff --git a/test/pc/e2e/peer_connection_e2e_smoke_test.cc b/test/pc/e2e/peer_connection_e2e_smoke_test.cc index a9e008b340..cb34fa0ed7 100644 --- a/test/pc/e2e/peer_connection_e2e_smoke_test.cc +++ b/test/pc/e2e/peer_connection_e2e_smoke_test.cc @@ -56,9 +56,6 @@ class PeerConnectionE2EQualityTestSmokeTest : public ::testing::Test { testing::UnitTest::GetInstance()->current_test_info()->name(), *network_emulation_->time_controller(), /*audio_quality_analyzer=*/nullptr, std::move(video_quality_analyzer)); - test::ScopedFieldTrials field_trials( - std::string(field_trial::GetFieldTrialString()) + - "WebRTC-UseStandardBytesStats/Enabled/"); } std::pair diff --git a/test/pc/e2e/peer_connection_quality_test.cc b/test/pc/e2e/peer_connection_quality_test.cc index 53e6220b7b..3345323d36 100644 --- a/test/pc/e2e/peer_connection_quality_test.cc +++ b/test/pc/e2e/peer_connection_quality_test.cc @@ -27,6 +27,7 @@ #include "pc/test/mock_peer_connection_observers.h" #include "rtc_base/gunit.h" #include "rtc_base/numerics/safe_conversions.h" +#include "rtc_base/strings/string_builder.h" #include "system_wrappers/include/cpu_info.h" #include "system_wrappers/include/field_trial.h" #include "test/pc/e2e/analyzer/audio/default_audio_quality_analyzer.h" @@ -63,6 +64,8 @@ constexpr TimeDelta kQuickTestModeRunDuration = TimeDelta::Millis(100); // Field trials to enable Flex FEC advertising and receiving. constexpr char kFlexFecEnabledFieldTrials[] = "WebRTC-FlexFEC-03-Advertised/Enabled/WebRTC-FlexFEC-03/Enabled/"; +constexpr char kUseStandardsBytesStats[] = + "WebRTC-UseStandardBytesStats/Enabled/"; class FixturePeerConnectionObserver : public MockPeerConnectionObserver { public: @@ -386,13 +389,19 @@ void PeerConnectionE2EQualityTest::Run(RunParams run_params) { void PeerConnectionE2EQualityTest::SetupRequiredFieldTrials( const RunParams& run_params) { - std::string field_trials = ""; + std::vector default_field_trials = { + kUseStandardsBytesStats}; if (run_params.use_flex_fec) { - field_trials += kFlexFecEnabledFieldTrials; + default_field_trials.push_back(kFlexFecEnabledFieldTrials); } - if (!field_trials.empty()) { - override_field_trials_ = std::make_unique( - field_trial::GetFieldTrialString() + field_trials); + if (!default_field_trials.empty()) { + rtc::StringBuilder sb; + sb << field_trial::GetFieldTrialString(); + for (const absl::string_view& field_trial : default_field_trials) { + sb << field_trial; + } + override_field_trials_ = + std::make_unique(sb.Release()); } }