1367562 - Deprecate non-standard frameRateMean and frameRateStdDev stats.

Status: RESOLVED FIXED

(Core :: WebRTC, task, P3)

Description

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

From Bug 1330587 comment 7. We should remove these now that we have framesEncoded.

Maybe land deprecation warnings, then open a second bug to remove in a release or two.

Comment 1

[Emma Humphries ☕️🎸🧞‍♀️✨ (she/they) [:emceeaich] (Pacific Time) use needinfo]

This is a P1 bug without an assignee. 

P1 are bugs which are being worked on for the current release cycle/iteration/sprint. 

If the bug is not assigned by Monday, 28 August, the bug's priority will be reset to '--'.

Comment 2

[Bulk Bug Changes for mreavy's org]

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

Comment 3

[Sylvestre Ledru [:Sylvestre]]

Moving to p3 because no activity for at least 1 year(s).
See https://github.com/mozilla/bug-handling/blob/master/policy/triage-bugzilla.md#how-do-you-triage for more information

Comment 4

[Andreas Pehrson [:pehrsons]]

Attachment: Bug 1367562 - Remove deprecated frame/bitrate stats. r?ng!,r?#webidl!

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/mozilla-central/rev/4de79de8b22b
Remove deprecated frame/bitrate stats. r=ng,webidl,smaug,fluent-reviewers,flod

Comment 6

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

https://hg.mozilla.org/mozilla-central/rev/4de79de8b22b

Comment 7

[Hamish Willee]

FF96 docs work for this can be tracked in https://github.com/mdn/content/issues/10858#issuecomment-986445621

Right now there doesn't seem to be any - none of the removed items appear to have been documented in MDN or in BCD.
Given that, all I plan to do is add a release note.

Further, there are some other changes to WebRTC that I found in https://groups.google.com/g/discuss-webrtc/c/pkAtJEF_unM

1. Can you please confirm that none of this has any web developer facing documentation requirement?

The only thing I though might was from the text that indicated: removal of bytesReceived on "remote-inbound-rtp" stats."

2. I found RTCInboundRtpStreamStats.bytesReceived and/or RTCIceCandidatePairStats.bytesReceived but these do not appear to be part of "remote-inbound-rtp" stats." - can you confirm?

Comment 8

[Andreas Pehrson [:pehrsons]]

(In reply to Hamish Willee from comment #7)

FF96 docs work for this can be tracked in https://github.com/mdn/content/issues/10858#issuecomment-986445621

Right now there doesn't seem to be any - none of the removed items appear to have been documented in MDN or in BCD.
Given that, all I plan to do is add a release note.

Further, there are some other changes to WebRTC that I found in https://groups.google.com/g/discuss-webrtc/c/pkAtJEF_unM

1. Can you please confirm that none of this has any web developer facing documentation requirement?

As part of bug 1654112 I most certainly think we should add a relnote for developers on this huge update. Those "sampling of improvements" bugs might not need any mention in relnotes (or docs -- they're bug fixes rather than features).

The only thing I though might was from the text that indicated: removal of bytesReceived on "remote-inbound-rtp" stats."

2. I found RTCInboundRtpStreamStats.bytesReceived and/or RTCIceCandidatePairStats.bytesReceived but these do not appear to be part of "remote-inbound-rtp" stats." - can you confirm?

I confirm.

Comment 9

[Hamish Willee]

Thanks. The bug https://bugzilla.mozilla.org/show_bug.cgi?id=1654112 has millions of things, few of which are tagged with docs needed. So while I think a release note would be awesome, I think your team will have to write something that explains the scope of the delivery. Personally I am only responsible for the notes on MDN itself, but I think Moz produces release notes and hacks posts in different ways.

Is there someone I can ask to summarize what has been delivered?

Comment 10

[Andreas Pehrson [:pehrsons]]

Ok yeah, I'll ping some people and see where we stand on this. I'm not sure whether there's something in motion already.

Comment 11

[Andreas Pehrson [:pehrsons]]

I'm told the wheels are in motion on this, so nothing further to do here.

Comment 12

[Hamish Willee]

Thanks Andreas. I've marked this as dev-doc-complete.