1371391 - Implement remaining fields in RTCIceCandidatePairStats

Status: NEW

(Core :: WebRTC: Networking, enhancement, P3)

Description

[Michael Froman [:mjf]]

Our implementation of RTCIceCandidatePairStats is still missing some of the fields from the spec:
https://w3c.github.io/webrtc-stats/#candidatepair-dict*

remaining:
  writeable
  readable
  totalRoundTripTime
  currentRoundTripTime
  availableOutgoingBitrate
  availableIncomingBitrate
  requestsReceived
  requestsSent
  responsesReceived
  responsesSent
  retransmissionsReceived
  retransmissionsSent
  consentRequestsSent

Comment 1

[Bulk Bug Changes for mreavy's org]

Mass change P2->P3 to align with new Mozilla triage process.

Comment 2

[Jim Mathies [:jimm]]

We implement spec gaps based on external partner needs. This has not been flagged by anyone and as such is low priority / backlog.

Comment 3

[Jim Mathies [:jimm]]

currentRoundTripTime would be the priority.

Comment 4

[Jan-Ivar Bruaroey [:jib] (needinfo? me)]

The mandatory to implement ones are https://w3c.github.io/webrtc-pc/#mandatory-to-implement-stats:

• transportId, localCandidateId, remoteCandidateId, state, nominated, bytesSent, bytesReceived, totalRoundTripTime, responsesReceived, currentRoundTripTime

Comment 5

[Michael Froman [:mjf]]

Attachment: Bug 1371391 - add rtt tracking for stun requests. r?bwc!

Comment 6

[Michael Froman [:mjf]]

Attachment: Bug 1371391 - surface rtt through ice_candidate_pair. r?bwc!

Comment 7

[Michael Froman [:mjf]]

Attachment: Bug 1371391 - plumb rtt through to NrIceCandidatePair. r?bwc!

Comment 8

[Michael Froman [:mjf]]

Attachment: Bug 1371391 - add remaining mandatory fields to RTCIceCandidatePairStats. r?bwc!

Comment 9

[Michael Froman [:mjf]]

Attachment: Bug 1371391 - populate new fields in RTCIceCandidatePairStats. r?bwc!