Fix references to getFirstEncoding in WebRTC tests

Bug: 986069, 1237909, 1410672
Change-Id: I0db8c7bc969682c812b6c40968e45c0e531e65c9
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4193280
Commit-Queue: Florent Castelli <orphis@chromium.org>
Auto-Submit: Florent Castelli <orphis@chromium.org>
Reviewed-by: Harald Alvestrand <hta@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1098512}

third_party/blink/web_tests/TestExpectations

@@ -5267,9 +5267,6 @@ crbug.com/1215390 [ Linux ] external/wpt/pointerevents/pointerevent_pointerId_sc
 crbug.com/1237640 http/tests/inspector-protocol/network/disabled-cache-navigation.js [ Pass Timeout ]
 crbug.com/1239175 http/tests/navigation/same-and-different-back.html [ Failure Pass ]
 crbug.com/1239164 http/tests/inspector-protocol/network/navigate-iframe-in2in.js [ Failure Pass ]
-crbug.com/1237909 external/wpt/webrtc-svc/RTCRtpParameters-scalability.html [ Crash Failure Pass Timeout ]
-crbug.com/1237909 [ Mac11-arm64 Release ] external/wpt/webrtc-svc/RTCRtpParameters-scalability-h264.html [ Crash Pass Timeout ]
-crbug.com/1237909 [ Mac12-arm64 Release ] external/wpt/webrtc-svc/RTCRtpParameters-scalability-h264.html [ Crash Pass Timeout ]
 
 # Sheriff 2021-08-19
 crbug.com/1234315 [ Mac ] virtual/threaded/external/wpt/animation-worklet/worklet-animation-with-scroll-timeline-and-overflow-hidden.https.html [ Failure Timeout ]

third_party/blink/web_tests/external/wpt/webrtc-extensions/RTCRtpParameters-adaptivePtime.html

@@ -6,12 +6,6 @@
 <script>
   'use strict';
 
-  function getFirstEncoding(param) {
-    const { encodings } = param;
-    assert_equals(encodings.length, 1);
-    return encodings[0];
-  }
-
   promise_test(async t => {
     const pc = new RTCPeerConnection();
     t.add_cleanup(() => pc.close());
@@ -20,14 +14,14 @@
     });
 
     let param = sender.getParameters();
-    let encoding = getFirstEncoding(param);
+    let encoding = param.encodings[0];
 
     assert_true(encoding.adaptivePtime);
 
     encoding.adaptivePtime = false;
     await sender.setParameters(param);
     param = sender.getParameters();
-    encoding = getFirstEncoding(param);
+    encoding = param.encodings[0];
 
     assert_false(encoding.adaptivePtime);
 
@@ -39,7 +33,7 @@
     const { sender } = pc.addTransceiver('audio', { sendEncodings: [{}] });
 
     const param = sender.getParameters();
-    const encoding = getFirstEncoding(param);
+    const encoding = param.encodings[0];
 
     assert_false(encoding.adaptivePtime);
 

third_party/blink/web_tests/external/wpt/webrtc-priority/RTCRtpParameters-encodings.html

@@ -25,7 +25,7 @@
 
     const param = sender.getParameters();
     validateSenderRtpParameters(param);
-    const encoding = getFirstEncoding(param);
+    const encoding = param.encodings[0];
 
     assert_equals(encoding.active, false);
     assert_equals(encoding.priority, 'low');

third_party/blink/web_tests/external/wpt/webrtc-svc/RTCRtpParameters-scalability.html

@@ -21,36 +21,26 @@
     });
 
     const param = sender.getParameters();
-    const encoding = getFirstEncoding(param);
+    const encoding = param.encodings[0];
 
     assert_equals(encoding.scalabilityMode, 'L1T3');
 
     encoding.scalabilityMode = 'L1T2';
     await sender.setParameters(param);
 
     const updatedParam = sender.getParameters();
-    const updatedEncoding = getFirstEncoding(updatedParam);
+    const updatedEncoding = updatedParam.encodings[0];
 
     assert_equals(updatedEncoding.scalabilityMode, 'L1T2');
   }, `Setting and updating scalabilityMode to a legal value should be accepted`);
 
-  promise_test(async t => {
-    const capabilities = RTCRtpSender.getCapabilities('video');
-    var svcSupported = false;
-    for (const codec of capabilities.codecs) {
-      if ('scalabilityModes' in codec && codec.scalabilityModes.length > 0) {
-        svcSupported = true;
-      }
-    }
-    assert_true(svcSupported);
-  }, `Sender capabilities should include at least some scalability modes`);
-
   promise_test(async t => {
     const pc = new RTCPeerConnection();
     t.add_cleanup(() => pc.close());
     const { sender } = pc.addTransceiver('video');
     const param = sender.getParameters();
-    assert_equals(param.encodings.length, 0);
+    const encoding = param.encodings[0];
+    assert_true(!('scalabilityMode' in encoding));
   }, 'Not setting sendEncodings results in no mode info before negotiation');
 
   promise_test(async t => {
@@ -60,29 +50,10 @@
       sendEncodings: [{}],
     });
     const param = sender.getParameters();
-    const encoding = getFirstEncoding(param);
-
+    const encoding = param.encodings[0];
     assert_true(!('scalabilityMode' in encoding));
   }, 'Not setting a scalability mode results in no mode set before negotiation');
 
-  promise_test(async t => {
-    const pc1 = new RTCPeerConnection();
-    const pc2 = new RTCPeerConnection();
-    t.add_cleanup(() => pc1.close());
-    t.add_cleanup(() => pc2.close());
-    const { sender } = pc1.addTransceiver('video', {
-      sendEncodings: [{}],
-    });
-    const param = sender.getParameters();
-    const encoding = getFirstEncoding(param);
-
-    exchangeIceCandidates(pc1, pc2);
-    await exchangeOfferAnswer(pc1, pc2);
-    const param2 = sender.getParameters();
-    const encoding2 = getFirstEncoding(param);
-    assert_true('scalabilityMode' in encoding2);
-  }, 'Not setting a scalability mode results in some mode set after negotiation');
-
   promise_test(async t => {
       const pc = new RTCPeerConnection();
     t.add_cleanup(() => pc.close());
@@ -105,7 +76,7 @@
     });
     // Before negotiation, the mode should be preserved.
     const param = transceiver.sender.getParameters();
-    const encoding = getFirstEncoding(param);
+    const encoding = param.encodings[0];
     assert_true('scalabilityMode' in encoding);
     // If L3T3 is not supported at all, abort test.
     assert_implements_optional(encoding.scalabilityMode === 'L3T3');