Closed Bug 1812358 Opened 3 years ago Closed 3 years ago

[wpt-sync] Sync PR 38172 - Fix references to getFirstEncoding in WebRTC tests

Categories

(Core :: WebRTC, task, P4)

Product:
Core
Component:
WebRTC
Type:
task

Tracking

()

Status:
RESOLVED FIXED
Milestone:
111 Branch
Tracking Flags:
Tracking Status
firefox111 --- fixed

People

(Reporter: wpt-sync, Unassigned)

References

(
URL
)

Details

(Whiteboard: [wptsync downstream])

Sync web-platform-tests PR 38172 into mozilla-central (this bug is closed when the sync is complete).

PR: https://github.com/web-platform-tests/wpt/pull/38172
Details from upstream follow.

Florent Castelli <orphis@chromium.org> wrote:

Fix references to getFirstEncoding in WebRTC tests

Bug: 986069
Change-Id: I0db8c7bc969682c812b6c40968e45c0e531e65c9

Reviewed-on: https://chromium-review.googlesource.com/4193280
WPT-Export-Revision: 3c62bf06279d02b64eaa0a67223ede5e7941fc7b

Component: web-platform-tests → WebRTC
Product: Testing → Core

CI Results

Ran 0 Firefox configurations based on mozilla-central, and Firefox, Chrome, and Safari on GitHub CI

Total 3 tests and 5 subtests

Status Summary

Firefox

OK : 3
PASS: 6
FAIL: 8

Chrome

OK : 3
PASS: 14

Safari

OK : 3
PASS: 8
FAIL: 6

Links

GitHub PR Head
GitHub PR Base

Details

Firefox-only Failures

  • /webrtc-priority/RTCRtpParameters-encodings.html [wpt.fyi]
    • sender.getParameters() should return sendEncodings set by addTransceiver(): FAIL
    • setParameters() with modified encoding.networkPriority should succeed with RTCRtpTransceiverInit: FAIL
    • setParameters() with modified encoding.networkPriority should succeed without RTCRtpTransceiverInit: FAIL

New Tests That Don't Pass

  • /webrtc-extensions/RTCRtpParameters-adaptivePtime.html [wpt.fyi]
    • Setting adaptivePtime should be accepted: FAIL (Chrome: PASS, Safari: FAIL)
    • adaptivePtime should be default false: FAIL (Chrome: PASS, Safari: FAIL)
  • /webrtc-priority/RTCRtpParameters-encodings.html [wpt.fyi]
    • sender.getParameters() should return sendEncodings set by addTransceiver(): FAIL (Chrome: PASS, Safari: PASS)
    • setParameters() with modified encoding.networkPriority should succeed with RTCRtpTransceiverInit: FAIL (Chrome: PASS, Safari: PASS)
    • setParameters() with modified encoding.networkPriority should succeed without RTCRtpTransceiverInit: FAIL (Chrome: PASS, Safari: PASS)
  • /webrtc-svc/RTCRtpParameters-scalability.html [wpt.fyi]
    • Setting and updating scalabilityMode to a legal value should be accepted: FAIL (Chrome: PASS, Safari: FAIL)
    • Setting a scalability mode to nonsense throws an exception: FAIL (Chrome: PASS, Safari: FAIL)
    • L3T3 on VP8 should return something other than L3T3: FAIL (Chrome: PASS, Safari: FAIL)
Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/e412e0b3baf2 [wpt PR 38172] - Fix references to getFirstEncoding in WebRTC tests, a=testonly

https://hg.mozilla.org/mozilla-central/rev/e412e0b3baf2

Status: NEW → RESOLVED
Closed: 3 years ago
status-firefox111: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 111 Branch
You need to log in before you can comment on or make changes to this bug.