From e2952a058e0b40848fa524eca668465b4b93a083 Mon Sep 17 00:00:00 2001 From: Harald Alvestrand Date: Thu, 19 Sep 2024 11:49:45 +0000 Subject: [PATCH] Eliminate a pointless IsEnabled helper Makes code easier to read. Bug: None Change-Id: I736f1a152101264184c3d7b1c7a72d1a3b022147 Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/362962 Reviewed-by: Emil Lundmark Commit-Queue: Harald Alvestrand Cr-Commit-Position: refs/heads/main@{#43051} --- media/engine/webrtc_voice_engine.cc | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/media/engine/webrtc_voice_engine.cc b/media/engine/webrtc_voice_engine.cc index 70e19fed00..69f2a3c811 100644 --- a/media/engine/webrtc_voice_engine.cc +++ b/media/engine/webrtc_voice_engine.cc @@ -247,10 +247,6 @@ std::optional ComputeSendBitrate(int max_send_bitrate_bps, } } -bool IsEnabled(const webrtc::FieldTrialsView& config, absl::string_view trial) { - return absl::StartsWith(config.Lookup(trial), "Enabled"); -} - struct AdaptivePtimeConfig { bool enabled = false; webrtc::DataRate min_payload_bitrate = webrtc::DataRate::KilobitsPerSec(16); @@ -473,9 +469,9 @@ WebRtcVoiceEngine::WebRtcVoiceEngine( apm_(audio_processing), audio_frame_processor_(std::move(audio_frame_processor)), minimized_remsampling_on_mobile_trial_enabled_( - IsEnabled(trials, "WebRTC-Audio-MinimizeResamplingOnMobile")), + trials.IsEnabled("WebRTC-Audio-MinimizeResamplingOnMobile")), payload_types_in_transport_trial_enabled_( - IsEnabled(trials, "WebRTC-PayloadTypesInTransport")) { + trials.IsEnabled("WebRTC-PayloadTypesInTransport")) { RTC_LOG(LS_INFO) << "WebRtcVoiceEngine::WebRtcVoiceEngine"; RTC_DCHECK(decoder_factory); RTC_DCHECK(encoder_factory);