Closed Bug 1367562 Opened 7 years ago Closed 3 years ago

Deprecate non-standard frameRateMean and frameRateStdDev stats.

Categories

(Core :: WebRTC, task, P3)

task

Tracking

()

RESOLVED FIXED
96 Branch
Tracking Status
firefox96 --- fixed

People

(Reporter: jib, Assigned: pehrsons)

References

(Depends on 1 open bug)

Details

(Keywords: dev-doc-complete, site-compat, stale-bug)

Attachments

(1 file)

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.
Rank: 18
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 '--'.
Keywords: stale-bug
Mass change P1->P2 to align with new Mozilla triage process
Priority: P1 → P2
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
Priority: P2 → P3
Type: enhancement → task
See Also: → 1341533
Assignee: nobody → apehrson
Status: NEW → ASSIGNED
Pushed by archaeopteryx@coole-files.de: https://hg.mozilla.org/mozilla-central/rev/4de79de8b22b Remove deprecated frame/bitrate stats. r=ng,webidl,smaug,fluent-reviewers,flod
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 96 Branch

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."

  1. I found RTCInboundRtpStreamStats.bytesReceived and/or RTCIceCandidatePairStats.bytesReceived but these do not appear to be part of "remote-inbound-rtp" stats." - can you confirm?
Flags: needinfo?(apehrson)

(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."

  1. 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.

Flags: needinfo?(apehrson) → needinfo?(hamishwillee)

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?

Flags: needinfo?(hamishwillee) → needinfo?(apehrson)

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.

Flags: needinfo?(apehrson)

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

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

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

Attachment

General

Created:
Updated:
Size: