- From: Henrik Boström <hbos@google.com>
- Date: Tue, 12 Feb 2019 14:05:20 +0100
- To: Elias Sun <elias.sun@ringcentral.com>
- Cc: "public-webrtc@w3.org" <public-webrtc@w3.org>, Patrice Khawam <patricek@ringcentral.com>, Fred Yang <fred.yang@ringcentral.com>, Edward Kogan <edk@ringcentral.com>, "na-g@nostrum.com" <na-g@nostrum.com>
- Message-ID: <CAEbRw2yANjb3cGrScqdOjj2qf6MT3=nPvRb4ApJTxrf5ohstmw@mail.gmail.com>
Hello! fractionLost is part of RTCRemoteInboundRtpStreamStats (remote-inbound-rtp), not RTCInboundRtpStreamStats (inbound-rtp). The mandatory list <https://w3c.github.io/webrtc-pc/#mandatory-to-implement-stats> (defined in webrtc-pc, not webrtc-stats) of this dictionary includes "all required attributes from its inherited dictionaries, and also attributes localId, roundTripTime". So currently it does not mandate fractionLost, and it sounds like the conformance <https://w3c.github.io/webrtc-stats/#conformance> section (of webrtc-stats) which recommends what other documents might mandate (without being authoratative?) is out-of-date. I see this issue <https://github.com/w3c/webrtc-stats/issues/388> has been filed on webrtc-stats. On Fri, Feb 8, 2019 at 9:31 AM Elias Sun <elias.sun@ringcentral.com> wrote: > Hi team, > > I filed a bug to the Firefox about the fractionLost is missed in the ' > inbound-rtp ‘. However, the guy in the Firefox team complains this field > is not in the dictionary RTCInboundRtpStreamStats > <https://w3c.github.io/webrtc-stats/#dom-rtcinboundrtpstreamstats> , so > the Firefox will not support it. However, the top of this web page has the > statement as below, > > An implementation *MUST* support generating statistics for the type > RTCInboundRtpStreamStats, with attributes packetsReceived, bytesReceived, > packetsLost, jitter, and fractionLost. > > The fractionLost is in the “MUST” support. But it is missed in the > dictionary below > > dictionary RTCInboundRtpStreamStats <https://w3c.github.io/webrtc-stats/#dom-rtcinboundrtpstreamstats> : RTCReceivedRtpStreamStats <https://w3c.github.io/webrtc-stats/#dom-rtcreceivedrtpstreamstats> { > DOMString <https://heycam.github.io/webidl/#idl-DOMString> trackId <https://w3c.github.io/webrtc-stats/#dom-rtcinboundrtpstreamstats-trackid>; > DOMString <https://heycam.github.io/webidl/#idl-DOMString> receiverId <https://w3c.github.io/webrtc-stats/#dom-rtcinboundrtpstreamstats-receiverid>; > DOMString <https://heycam.github.io/webidl/#idl-DOMString> remoteId <https://w3c.github.io/webrtc-stats/#dom-rtcinboundrtpstreamstats-remoteid>; > unsigned long <https://heycam.github.io/webidl/#idl-unsigned-long> framesDecoded <https://w3c.github.io/webrtc-stats/#dom-rtcinboundrtpstreamstats-framesdecoded>; > DOMHighResTimeStamp lastPacketReceivedTimestamp <https://w3c.github.io/webrtc-stats/#dom-rtcinboundrtpstreamstats-lastpacketreceivedtimestamp>; > double <https://heycam.github.io/webidl/#idl-double> averageRtcpInterval <https://w3c.github.io/webrtc-stats/#dom-rtcinboundrtpstreamstats-averagertcpinterval>; > unsigned long <https://heycam.github.io/webidl/#idl-unsigned-long> fecPacketsReceived <https://w3c.github.io/webrtc-stats/#dom-rtcinboundrtpstreamstats-fecpacketsreceived>; > unsigned long long <https://heycam.github.io/webidl/#idl-unsigned-long-long> bytesReceived <https://w3c.github.io/webrtc-stats/#dom-rtcinboundrtpstreamstats-bytesreceived>; > unsigned long <https://heycam.github.io/webidl/#idl-unsigned-long> packetsFailedDecryption <https://w3c.github.io/webrtc-stats/#dom-rtcinboundrtpstreamstats-packetsfaileddecryption>; > unsigned long <https://heycam.github.io/webidl/#idl-unsigned-long> packetsDuplicated <https://w3c.github.io/webrtc-stats/#dom-rtcinboundrtpstreamstats-packetsduplicated>; > record <https://heycam.github.io/webidl/#idl-record><USVString <https://heycam.github.io/webidl/#idl-USVString>, unsigned long <https://heycam.github.io/webidl/#idl-unsigned-long>> perDscpPacketsReceived <https://w3c.github.io/webrtc-stats/#dom-rtcinboundrtpstreamstats-perdscppacketsreceived>; > }; > > > > Please add the fractionLost into the dictionary RTCInboundRtpStreamStats > officially. > > By the way, Chrome and Safari support it. > > Thanks, > > Elias > > >
Received on Tuesday, 12 February 2019 13:05:55 UTC