commit 6e23fa52bf66aa581269316683cd8b138fe70afc
author Philipp Hancke <phancke@microsoft.com> Mon May 22 16:27:45 2023
committer WebRTC LUCI CQ <webrtc-scoped@luci-project-accounts.iam.gserviceaccount.com> Thu May 25 19:25:07 2023
tree 13920c1f3c0d4328c13df2f071fdd3bf981c24a1
parent 56d126074e5fb62a65e4e14cce44466ce7297770

Cleanup WebRTC-PayloadTypes-Lower-Dynamic-Range trial

as the killswitch is no longer required.

BUG=webrtc:12194

Change-Id: Icb825012c50a93ec4dae49be5732d9e4c0adb89d
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/306182
Reviewed-by: Henrik Boström <hbos@webrtc.org>
Reviewed-by: Florent Castelli <orphis@webrtc.org>
Commit-Queue: Philipp Hancke <phancke@microsoft.com>
Cr-Commit-Position: refs/heads/main@{#40149}

media/base/codec.cc

diff --git a/media/base/codec.cc b/media/base/codec.cc
index e43d61c..7ce7947 100644
--- a/media/base/codec.cc
+++ b/media/base/codec.cc
@@ -135,18 +135,11 @@
                     const webrtc::FieldTrialsView* field_trials) const {
   // Match the codec id/name based on the typical static/dynamic name rules.
   // Matching is case-insensitive.
-
-  // Legacy behaviour with killswitch.
-  if (field_trials &&
-      field_trials->IsDisabled("WebRTC-PayloadTypes-Lower-Dynamic-Range")) {
-    const int kMaxStaticPayloadId = 95;
-    return (id <= kMaxStaticPayloadId || codec.id <= kMaxStaticPayloadId)
-               ? (id == codec.id)
-               : (absl::EqualsIgnoreCase(name, codec.name));
-  }
   // We support the ranges [96, 127] and more recently [35, 65].
   // https://www.iana.org/assignments/rtp-parameters/rtp-parameters.xhtml#rtp-parameters-1
   // Within those ranges we match by codec name, outside by codec id.
+  // Since no codecs are assigned an id in the range [66, 95] by us, these will
+  // never match.
   const int kLowerDynamicRangeMin = 35;
   const int kLowerDynamicRangeMax = 65;
   const int kUpperDynamicRangeMin = 96;