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.

Attachment

General

Created:
Updated:
Size: