Bug 267696
| Summary: | Missing WebRTC stats in Safari | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Kaustav <kagh> |
| Component: | WebRTC | Assignee: | youenn fablet <youennf> |
| Status: | RESOLVED FIXED | ||
| Severity: | Critical | CC: | dibotero, kagh, webkit-bug-importer, youennf |
| Priority: | P2 | Keywords: | InRadar |
| Version: | Safari 17 | ||
| Hardware: | All | ||
| OS: | All | ||
| Bug Depends on: | |||
| Bug Blocks: | 269314 | ||
Kaustav
These following important WebRTC stats are missing in Safari. Could you please add these?
inbound-rtp: qpSum, rtxSsrc, fecSsrc, decoderImplementation
outbound-rtp: rtxSsrc, scalabilityMode, encoderImplementation
data-channel: dataChannelIdentifier
local-candidate: foundation, usernameFragment, tcpType
remote-candidate: foundation, usernameFragment, tcpType
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Kaustav
two more stats
local-candidate: relayProtocol
remote-candidate: relayProtocol
[MSFT] Diego Perez Botero
Bonus points for the rest of the FEC stats (already implemented in Chromium's libWebRTC)
inbound-rtp: fecBytesReceived, fecPacketsReceived, fecPacketsDiscarded
Radar WebKit Bug Importer
<rdar://problem/121594743>
youenn fablet
For qpSum and scalabilityMode, we need to pipe the info of the remote decoder/encoder to libwebrtc. I filed https://bugs.webkit.org/show_bug.cgi?id=268155 to keep track of this one.
I do not think we want to expose decoderImplementation/encoderImplementation at this point. Media Capabilities can be used to check whether it should be powerEfficient or not, which should in general match whether VTB is used or not.
youenn fablet
Pull request: https://github.com/WebKit/WebKit/pull/23300
EWS
Committed 273643@main (7b5229864801): <https://commits.webkit.org/273643@main>
Reviewed commits have been landed. Closing PR #23300 and removing active labels.