commit a5f895a366a97e1c22c142e83c1f34d0c899a92b
author Danil Chapovalov <danilchap@webrtc.org> Tue Apr 16 09:41:56 2024
committer WebRTC LUCI CQ <webrtc-scoped@luci-project-accounts.iam.gserviceaccount.com> Tue Apr 16 10:38:37 2024
tree 22178d47b01eddc5b7c31786831f1e921070ef84
parent 501c4f37bfee47b26999ee291c5355ad64554df7

Delete field trial WebRTC-UseShortVP8TL2Pattern as unused

Bug: webrtc:9477, webrtc:11503
Change-Id: I65551a00c394aa39b0d30ecd343616e8142d1df1
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/347522
Reviewed-by: Erik Språng <sprang@webrtc.org>
Commit-Queue: Danil Chapovalov <danilchap@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#42082}

experiments/field_trials.py

diff --git a/experiments/field_trials.py b/experiments/field_trials.py
index 43e2e73..5e3521c 100755
--- a/experiments/field_trials.py
+++ b/experiments/field_trials.py
@@ -781,9 +781,6 @@
     FieldTrial('WebRTC-UseDifferentiatedCellularCosts',
                'webrtc:11473',
                date(2024, 4, 1)),
-    FieldTrial('WebRTC-UseShortVP8TL2Pattern',
-               'webrtc:9477',
-               INDEFINITE),
     FieldTrial('WebRTC-UseShortVP8TL3Pattern',
                'webrtc:8162',
                INDEFINITE),
@@ -908,7 +905,7 @@
 ])  # yapf: disable
 
 POLICY_EXEMPT_FIELD_TRIALS_DIGEST: str = \
-    '2387f9c58686fb4e4f9baa69b9323ef857255d6a'
+    'c8a81e2e8f6e55640a1093bddb8efdf668341e2a'
 
 REGISTERED_FIELD_TRIALS: FrozenSet[FieldTrial] = ACTIVE_FIELD_TRIALS.union(
     POLICY_EXEMPT_FIELD_TRIALS)

modules/video_coding/codecs/vp8/default_temporal_layers.cc

diff --git a/modules/video_coding/codecs/vp8/default_temporal_layers.cc b/modules/video_coding/codecs/vp8/default_temporal_layers.cc
index 94860da..8ad30f0 100644
--- a/modules/video_coding/codecs/vp8/default_temporal_layers.cc
+++ b/modules/video_coding/codecs/vp8/default_temporal_layers.cc
@@ -141,29 +141,13 @@
       // that the 'alt' buffer reference is effectively the last keyframe.
       // TL0 also references and updates the 'last' buffer.
       // TL1 also references 'last' and references and updates 'golden'.
-      if (!field_trial::IsDisabled("WebRTC-UseShortVP8TL2Pattern")) {
-        // Shortened 4-frame pattern:
-        //   1---1   1---1 ...
-        //  /   /   /   /
-        // 0---0---0---0 ...
-        return {{"SS", {kReferenceAndUpdate, kNone, kNone}},
-                {"-S", {kReference, kUpdate, kNone}},
-                {"SR", {kReferenceAndUpdate, kNone, kNone}},
-                {"-D", {kReference, kReference, kNone, kFreezeEntropy}}};
-      } else {
-        // "Default" 8-frame pattern:
-        //   1---1---1---1   1---1---1---1 ...
-        //  /   /   /   /   /   /   /   /
-        // 0---0---0---0---0---0---0---0 ...
-        return {{"SS", {kReferenceAndUpdate, kNone, kNone}},
-                {"-S", {kReference, kUpdate, kNone}},
-                {"SR", {kReferenceAndUpdate, kNone, kNone}},
-                {"-R", {kReference, kReferenceAndUpdate, kNone}},
-                {"SR", {kReferenceAndUpdate, kNone, kNone}},
-                {"-R", {kReference, kReferenceAndUpdate, kNone}},
-                {"SR", {kReferenceAndUpdate, kNone, kNone}},
-                {"-D", {kReference, kReference, kNone, kFreezeEntropy}}};
-      }
+      //   1---1   1---1 ...
+      //  /   /   /   /
+      // 0---0---0---0 ...
+      return {{"SS", {kReferenceAndUpdate, kNone, kNone}},
+              {"-S", {kReference, kUpdate, kNone}},
+              {"SR", {kReferenceAndUpdate, kNone, kNone}},
+              {"-D", {kReference, kReference, kNone, kFreezeEntropy}}};
     case 3:
       if (field_trial::IsEnabled("WebRTC-UseShortVP8TL3Pattern")) {
         // This field trial is intended to check if it is worth using a shorter
@@ -708,11 +692,7 @@
     case 1:
       return {{0}};
     case 2:
-      if (!field_trial::IsDisabled("WebRTC-UseShortVP8TL2Pattern")) {
-        return {{2}, {0}, {0}, {1, 2}};
-      } else {
-        return {{6}, {0}, {0}, {1, 2}, {2}, {3, 4}, {4}, {5, 6}};
-      }
+      return {{2}, {0}, {0}, {1, 2}};
     case 3:
       if (field_trial::IsEnabled("WebRTC-UseShortVP8TL3Pattern")) {
         return {{0}, {0}, {0}, {0, 1, 2}};