Re: [webrtc-stats] Reuse of "inbound-rtp" and "outbound-rtp" for RTCP is confusing.

`isRemote` would be falsy once removed, so finding *local* stats wouldn't break.

For *remote* stats in Firefox, I hope most users obtain it through the `remoteId` (aka `associateStatsId` in the spec) shown in my [first comment](https://github.com/w3c/webrtc-stats/issues/189#issue-216190363). If so, then this change won't affect them. If they instead try to enumerate for the remote stats the old way, then the new type names mean they won't find them. Not great, but beats finding rtcp data and thinking it is local data.

(Keeping `remoteId` as an alias for `associateStatsId` for a good while is a separate issue, but a simple thing for us to do. We'll make it non-enumerable so it won't show up but still work.)

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

Received on Thursday, 30 March 2017 18:43:17 UTC