Re: [webrtc-stats] Adding "codec type" and transportId to RTCCodecStats.

I think the question we need to answer is: "what object does an `RTCCodecStats` represent"? This is missing from the description of `RTCCodecStats` right now.

We could say it represents an SDP media format, or it represents an instance of an encoder/decoder. The fact that it has an `implementation` field made me think it was the latter.

Or maybe it *was* intended to be just an SDP media format, and `implementation` is the abnormality. But I can imagine adding things like "encode/decode time in milliseconds" or "encode/decode CPU utilization" (which are stats Chrome collects, but doesn't expose). In which case the "instance of encoder/decoder" interpretation would make more sense.

-- 
GitHub Notification of comment by taylor-b
Please view or discuss this issue at https://github.com/w3c/webrtc-stats/pull/195#issuecomment-297791882 using your GitHub account

Received on Thursday, 27 April 2017 17:59:35 UTC