W3C home > Mailing lists > Public > public-webrtc-logs@w3.org > June 2022

Re: [webrtc-provisional-stats] Add sender/receiver/transceiver to provisional spec (#32)

From: henbos via GitHub <sysbot+gh@w3.org>
Date: Thu, 23 Jun 2022 09:53:10 +0000
To: public-webrtc-logs@w3.org
Message-ID: <issue_comment.created-1164198553-1655977988-sysbot+gh@w3.org>
I merged PR #34 as previously discussed, which added back the sender/receiver/transceiver dictionaries to the provisional spec (see https://w3c.github.io/webrtc-provisional-stats/#RTCRtpTransceiverStats-dict*).

But before I close this issue, please take a look @jan-ivar @vr000m @alvestrand.

I want to be perfectly clear that when I added back sender and receiver I did _not_ have these dictionaries extend `RTCMediaStreamTrackStats` so this is technically not just editorial.

The reason I did not make them extend "track" is because that dictionary is obsolete. We could certainly have them extend "track" stats in order for this to be a purely editorial change (let me know and I will create a PR) but the reason I did not do this is because a) nobody ever implemented this and it would only serve to document git history, and b) having sender extend track is horribly under-specified and makes absolutely no sense for simulcast stats. It seems like a silly maintenance burden to have the sender extend the obsolete track in a provisional stats spec, and not extending receiver with track stats is the consistent thing to do.

Let me know if I can close this.

GitHub Notification of comment by henbos
Please view or discuss this issue at https://github.com/w3c/webrtc-provisional-stats/issues/32#issuecomment-1164198553 using your GitHub account

Sent via github-notify-ml as configured in https://github.com/w3c/github-notify-ml-config
Received on Thursday, 23 June 2022 09:53:12 UTC

This archive was generated by hypermail 2.4.0 : Saturday, 6 May 2023 21:19:57 UTC