From d621b419a3cb6d7c66ebdbf5bb2f62e283c02864 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Erik=20Spr=C3=A5ng?= Date: Mon, 20 Jan 2025 15:01:38 +0100 Subject: [PATCH] Make WebRTC-Video-AV1EvenPayloadSizes default-on. MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Experimentation shows only a slight increase in bitrate due to improved BWE. There's no negative side-effects we have been able to see so far. This CL flips the experiment to default-on but is kept around as a kill-switch until the next milestone just in case something unexpected is discovered. Bug: webrtc:42226301 Change-Id: I4a0b1c85e912b909d7bff58d78966cf161857f7c Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/374182 Reviewed-by: Danil Chapovalov Auto-Submit: Erik Språng Commit-Queue: Erik Språng Cr-Commit-Position: refs/heads/main@{#43773} --- modules/rtp_rtcp/source/rtp_sender_video.cc | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/modules/rtp_rtcp/source/rtp_sender_video.cc b/modules/rtp_rtcp/source/rtp_sender_video.cc index 8a21d0523f..248658ef4f 100644 --- a/modules/rtp_rtcp/source/rtp_sender_video.cc +++ b/modules/rtp_rtcp/source/rtp_sender_video.cc @@ -191,8 +191,9 @@ RTPSenderVideo::RTPSenderVideo(const Config& config) rtp_sender_->SSRC(), config.task_queue_factory) : nullptr), - enable_av1_even_split_( - config.field_trials->IsEnabled("WebRTC-Video-AV1EvenPayloadSizes")) { + // TODO: bugs.webrtc.org/42226301 - Remove when fully launched. + enable_av1_even_split_(!config.field_trials->IsDisabled( + "WebRTC-Video-AV1EvenPayloadSizes")) { if (frame_transformer_delegate_) frame_transformer_delegate_->Init(); }