commit 9146d765e20cc2b1d599b67ed875629cf43771e6
author Evan Shrubsole <eshr@webrtc.org> Wed Oct 13 11:19:48 2021 +0200
committer WebRTC LUCI CQ <webrtc-scoped@luci-project-accounts.iam.gserviceaccount.com> Wed Oct 13 17:01:14 2021 +0000
tree 9d81d63f4c0641eeb353d9abdac883d8fbaba724
parent af1038d97c566c0f7ad12f9dd31b2f699833143d

Cleaup WebRTC-AddRttToPlayoutDelay field trial

Field trial is not used in any rollouts and should be removed.

R=mhoro@webrtc.org

Bug: webrtc:13264
Change-Id: Ib896dcdec81db7c3f4e68a8dda266d96dfdc6aed
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/234865
Reviewed-by: Philip Eliasson <philipel@webrtc.org>
Reviewed-by: Michael Horowitz <mhoro@webrtc.org>
Commit-Queue: Evan Shrubsole <eshr@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#35195}

modules/video_coding/frame_buffer2_unittest.cc

diff --git a/modules/video_coding/frame_buffer2_unittest.cc b/modules/video_coding/frame_buffer2_unittest.cc
index f2a0589..129c25f 100644
--- a/modules/video_coding/frame_buffer2_unittest.cc
+++ b/modules/video_coding/frame_buffer2_unittest.cc
@@ -140,8 +140,7 @@
   static constexpr size_t kFrameSize = 10;
 
   TestFrameBuffer2()
-      : trial_("WebRTC-AddRttToPlayoutDelay/Enabled/"),
-        time_controller_(Timestamp::Seconds(0)),
+      : time_controller_(Timestamp::Seconds(0)),
         time_task_queue_(
             time_controller_.GetTaskQueueFactory()->CreateTaskQueue(
                 "extract queue",
@@ -232,8 +231,6 @@
 
   uint32_t Rand() { return rand_.Rand<uint32_t>(); }
 
-  // The ProtectionMode tests depends on rtt-multiplier experiment.
-  test::ScopedFieldTrials trial_;
   webrtc::GlobalSimulatedTimeController time_controller_;
   rtc::TaskQueue time_task_queue_;
   VCMTimingFake timing_;
@@ -459,28 +456,6 @@
   EXPECT_LT(timing_.GetCurrentJitter(), kRttMs);
 }
 
-TEST_F(TestFrameBuffer2, ProtectionModeNack) {
-  uint16_t pid = Rand();
-  uint32_t ts = Rand();
-  constexpr int64_t kRttMs = 200;
-
-  buffer_->UpdateRtt(kRttMs);
-
-  // Jitter estimate includes RTT (after 3 retransmitted packets)
-  buffer_->SetProtectionMode(kProtectionNack);
-  InsertNackedFrame(pid, ts);
-  InsertNackedFrame(pid + 1, ts + 100);
-  InsertNackedFrame(pid + 2, ts + 200);
-  InsertFrame(pid + 3, 0, ts + 300, true, kFrameSize);
-  ExtractFrame();
-  ExtractFrame();
-  ExtractFrame();
-  ExtractFrame();
-  ASSERT_EQ(4u, frames_.size());
-
-  EXPECT_GT(timing_.GetCurrentJitter(), kRttMs);
-}
-
 TEST_F(TestFrameBuffer2, NoContinuousFrame) {
   uint16_t pid = Rand();
   uint32_t ts = Rand();