- 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