From 38de6bc0b8444a0f84034d5b684d141bd00e5001 Mon Sep 17 00:00:00 2001 From: Florent Castelli Date: Tue, 30 Aug 2022 13:06:49 +0200 Subject: [PATCH] svc: Remove use of the VideoFrameTrackingIdAdvertised trial AV1 tests seem to be running fine now that we have the dependency descriptor enabled, so remove the need for the RTP header extension as it doesn't allow discarding frames. Bug: webrtc:11607 Change-Id: Ifd0670ab61a5b69d0570f65ba30c352a31376992 Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/273488 Reviewed-by: Artem Titov Auto-Submit: Florent Castelli Commit-Queue: Florent Castelli Cr-Commit-Position: refs/heads/main@{#37952} --- pc/test/svc_e2e_tests.cc | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/pc/test/svc_e2e_tests.cc b/pc/test/svc_e2e_tests.cc index d33d2862aa..fe25232d72 100644 --- a/pc/test/svc_e2e_tests.cc +++ b/pc/test/svc_e2e_tests.cc @@ -244,9 +244,7 @@ MATCHER_P2(HasSpatialAndTemporalLayers, } TEST_P(SvcTest, ScalabilityModeSupported) { - // Track frames using an RTP header instead of modifying the encoded data as - // this doesn't seem to work for AV1. - std::string trials = "WebRTC-VideoFrameTrackingIdAdvertised/Enabled/"; + std::string trials; if (UseDependencyDescriptor()) { trials += "WebRTC-DependencyDescriptorAdvertised/Enabled/"; }