W3C home > Mailing lists > Public > public-webrtc-logs@w3.org > April 2017

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>
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
Received on Monday, 3 April 2017 21:32:40 UTC

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