Closed Bug 1714303 Opened 3 years ago Closed 3 years ago

Update various things about Media Playback and WebRTC telemetry probes

Categories

(Core :: Audio/Video, task)

task

Tracking

()

RESOLVED FIXED
91 Branch
Tracking Status
firefox91 --- fixed

People

(Reporter: padenot, Assigned: padenot)

References

Details

Attachments

(6 files)

  • Remove Nils and Jean-Yves, adding myself, alwu, bryce and pehrsons instead, with the following loose rule:
    • EME: alwu, bryce
    • playback: bryce, myself, alwu but not always
    • autoplay: alwu, myself
    • mediarecorder: bryrce, pehrsons
    • WebRTC stuff: there is an alias, just removed Nils there
    • Web Audio: myself
  • Bump some probes to version 100. Some have expired for some time, but because of the work we're going to do on performance and energy savings, I'd like to have them back. Also for the tab indicator click, I'd like to have some data because of the redesign
  • Add this bug to the array
Assignee: nobody → padenot
Status: NEW → ASSIGNED

Depends on D116697

Request for Data Collection Renewal

** This form is for the renewal of an existing, reviewed data collection.**

** All questions are mandatory.
You must receive Data Review from a
Data Steward
on a filled-out Request before shipping your renewed data collection.**

  1. Provide a link to the initial Data Collection Review Request for this collection.

WEBRTC_DTLS_CIPHER: https://bugzilla.mozilla.org/show_bug.cgi?id=1484024#c6
WEBRTC_SRTP_CIPHER: https://bugzilla.mozilla.org/show_bug.cgi?id=1491511#c4
MEDIA_RECORDER_RECORDING_DURATION, MEDIA_RECORDER_TRACK_ENCODER_INIT_TIMEOUT_TYPE: https://bugzilla.mozilla.org/show_bug.cgi?id=1400757#c2
VIDEO_INFERRED_DECODE_SUSPEND_PERCENTAGE: https://bugzilla.mozilla.org/show_bug.cgi?id=1293145#c3
VIDEO_EME_REQUEST_SUCCESS_LATENCY_MS, VIDEO_EME_REQUEST_FAILURE_LATENCY_MS: introduced in 1187760, no data-review
TAB_AUDIO_INDICATOR_USED: https://bugzilla.mozilla.org/show_bug.cgi?id=1314220#c11
VIDEO_UNBLACKINGLISTING_DXVA_DRIVER_RUNTIME_STATUS: introduced in https://bugzilla.mozilla.org/show_bug.cgi?id=1393392, no data-review
WEB_AUDIO_BECOMES_AUDIBLE_TIME: https://bugzilla.mozilla.org/show_bug.cgi?id=1490074#c3

  1. When will this collection now expire?

All in 100

  1. Why was the initial period of collection insufficient?

A rationale for each of those:

WEBRTC_DTLS_CIPHER, WEBRTC_SRTP_CIPHER

This is still needed to monitor what server-side components are doing. The WebRTC might ask for a permanent probe.

MEDIA_RECORDER_RECORDING_DURATION

we though very little people used the MediaRecorder API and didn't maintain it much, but that's incorrect based on crash data. We might ask for a permanent probe next time.

MEDIA_RECORDER_TRACK_ENCODER_INIT_TIMEOUT_TYPE

We're supporting hardware encoder now, and we'd like to understand when it fails (it's much more prone to fail than software encoders).

VIDEO_INFERRED_DECODE_SUSPEND_PERCENTAGE

Other teams are looking into energy efficiency of Firefox. This is one of the probes to support this effort: video decoding is extremely power-hungry, and making sure our suspend mechanism works in the wild is essential. This might need to become a permanent probe.

VIDEO_EME_REQUEST_SUCCESS_LATENCY_MS, VIDEO_EME_REQUEST_FAILURE_LATENCY_MS

Simple performance monitoring, this number should generally stay stable.

TAB_AUDIO_INDICATOR_USED

MR1 saw a massive change there: it's now only visible on hover. This is good for monitoring discoverability.

VIDEO_UNBLACKINGLISTING_DXVA_DRIVER_RUNTIME_STATUS

This monitors re-enabling of hardware accelerated video decoding in case of crash. I believe it might need to become permanent, as graphics drivers are constantly moving between being allowed and disallowed, and hardware decoding is critical to remain competitive (and save power, or just play things like 4k videos smoothly).

WEB_AUDIO_BECOMES_AUDIBLE_TIME

We still haven't shipped blocking the Web Audio API from automatically playing, and this probe is essential to understand if it's feasible. Additionally, and to further support the energy efficiency support, we'd like to understand how often an AudioContext from the Web Audio API is instantiated but not used, because this has a big energy consumption impact (it's one of my personal tasks for H2).

In addition to all of this WEBRTC_DTLS_PROTOCOL_VERSION is removed, it's not useful anymore.

Attached file renewal_request.md

Or maybe it must be an attachment, I can never remember. Same text in both though, except one difference for MEDIA_RECORDER_RECORDING_DURATION that is here.

Attachment #9226356 - Flags: data-review?(chutten)

Comment on attachment 9226356 [details]
renewal_request.md

Excellent rationale explanations, thank you!

Is the provided Data Collection Review complete, correct, and data-review+ by a Data Steward?

Yes.

Is the data collection covered by the existing Firefox Privacy Notice?

Yes.


Result: datareview+

Attachment #9226356 - Flags: data-review?(chutten) → data-review+
Attachment #9224909 - Attachment description: Bug 1714303 - Bump some telemetry probes to 100. r?bryce → Bug 1714303 - Bump some telemetry probes to 100. r?bryce data-review=chutten
Pushed by padenot@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/334dfe8008c5
Update emails for media telemetry probes. r=alwu,bryce,ng
https://hg.mozilla.org/integration/autoland/rev/3bd3b868ab68
Bump some telemetry probes to 100. r=bryce data-review=chutten
https://hg.mozilla.org/integration/autoland/rev/47a0a6429b4e
Add this bug number's to all probes touched. r=bryce
https://hg.mozilla.org/integration/autoland/rev/6724f5124355
Remove the probe about chained ogg. r=bryce
https://hg.mozilla.org/integration/autoland/rev/e791bc86271d
Remove WEBRTC_DTLS_PROTOCOL_VERSION. r=bwc
See Also: → 1754644
See Also: → 1754645
See Also: → 1754647
See Also: → 1754649
See Also: → 1754651
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: