Re: [webrtc-stats] WPT tests are wrong about when "outbound-rtp" and "inbound-rtp" stats appear (#619)

We got rid of "sender", "receiver", "transceiver" and "track" stats in favor of just moving everything to "outbound-rtp" and "inbound-rtp" which we've already done in both spec and implementation so I think delaying the creation of the RTP objects is harmful.

On top of that, I don't like the idea of permanent objects since a) it's a memory leak, b) it adds implementation complexity, and c) it rarely seems useful; in practise people poll getStats periodically during a call, not a single time after the call has already ended.

So to me it seems easier to simplify the spec than to insist that the implementation delays creation of stats objects that you would legitimately want to be there.

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


-- 
Sent via github-notify-ml as configured in https://github.com/w3c/github-notify-ml-config

Received on Friday, 17 June 2022 09:52:38 UTC