Closed Bug 1531087 Opened 5 years ago Closed 1 year ago

RTCPeerConnectionStats ("peer-connection" RTCStatsType) support

Categories

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

enhancement

Tracking

()

RESOLVED FIXED
113 Branch
Tracking Status
firefox113 --- fixed

People

(Reporter: bwc, Assigned: bwc)

References

(Blocks 3 open bugs, Regressed 1 open bug)

Details

(4 keywords)

Attachments

(1 file)

No description provided.
Summary: RTCPeerConnectionStats support → RTCPeerConnectionStats ("peer-connection" RTCStatsType) support

We ought to think about whether we need to document all the stats types somewhere.

Keywords: dev-doc-needed

We ought to think about whether we need to document all the stats types somewhere.

I'd recommend starting with the mandatory to implement subset.

Blocks: 1533020

web.skype.com need this:
TypeError: RTCPeerConnection.getStats: Argument 1 does not implement interface MediaStreamTrack.

Severity: normal → S3
Blocks: 1814252

Depends on D168433

Assignee: nobody → docfaraday
Status: NEW → ASSIGNED

Try looks fine.

Currently relies on the patches from bug 1813147; could possibly rebase, but as long as we can get bug 1813147 done, no need to do that.

Depends on: 1813147
Blocks: 1278299
Pushed by bcampen@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/0bf440baea51
Implement RTCPeerConnectionStats. r=ng,webidl,smaug

Backed out for causing build bustages on PeerConnectionImpl.cpp

mochitest log: https://treeherder.mozilla.org/logviewer?job_id=405685822&repo=autoland

Flags: needinfo?(docfaraday)
Pushed by bcampen@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/3bcc93903fe1
Implement RTCPeerConnectionStats. r=ng,webidl,smaug
Pushed by bcampen@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/af84cffee628
Implement RTCPeerConnectionStats. r=ng,webidl,smaug
Regressions: 1822350
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 113 Branch
Flags: needinfo?(docfaraday)

FYI MDN docs for this can be tracked inhttps://github.com/mdn/content/issues/26146

The docs and compatibility work is done, but not yet reviewed.

You need to log in before you can comment on or make changes to this bug.