Re: [webrtc-stats] New 'remote-inbound-rtp' + 'remote-outbound-rtp' (refactor out isRemote)

From: jan-ivar via GitHub <sysbot+gh@w3.org> Date: Mon, 03 Apr 2017 21:32:33 +0000 To: public-webrtc-logs@w3.org Message-ID: <issue_comment.created-291280562-1491255152-sysbot+gh@w3.org> · This archive was generated by hypermail 2.4.0 : Saturday, 6 May 2023 21:19:40 UTC

Note that this PR also adds back `remoteId` instead of `associateStatsId`, because this is now possible. It also adds a symmetrical `localId`. I think this will help avoid bugs like the one in the stats example.

I found it also helps with readability in the spec, since we can now read the exact corresponding type that `remoteId`/`localId` point to, rather than the nondescript `RTCStats`. [Preview here](https://rawgit.com/jan-ivar/webrtc-stats/197e58083aa0d113ec84866f591ceb9933b2c0e0/webrtc-stats.html#inboundrtpstats-dict*).

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