Closed Bug 1602163 Opened 4 years ago Closed 4 years ago

VideoPlaybackQuality corruptedFrames deprecated

Categories

(Core :: Audio/Video: Playback, enhancement)

enhancement
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla73
Tracking Status
firefox73 --- fixed

People

(Reporter: chcunningham, Assigned: alwu)

Details

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

Attachments

(2 files)

User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_14_6) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/78.0.3904.108 Safari/537.36

Steps to reproduce:

VideoPlaybackQuality initially shipped with a corruptedFrames attribute. Media WG resolved to remove it from the spec.

https://github.com/w3c/media-playback-quality/pull/20

This issue tracks updates to Firefox to remove it from the API.

Note: While most implementers simply returned 0, some sites out there may still be expecting this attribute to exist (ex: if only to log it). We are just now shipping the VideoPlaybackQuality API in Chrome, so we don't have telemetry on usage of corruptedFrames. Our plan is to collect some metrics and potentially deprecate this slowly.

Chrome ship thread:
https://groups.google.com/a/chromium.org/d/msg/blink-dev/fgE0Ua9ZSU8/3XM9bU1LAgAJ

Status: UNCONFIRMED → NEW
Component: Untriaged → Audio/Video: Playback
Ever confirmed: true
Flags: needinfo?(gsquelart)
Product: Firefox → Core

Alistair, is this something you could take over?

Flags: needinfo?(gsquelart) → needinfo?(alwu)
Assignee: nobody → alwu
Flags: needinfo?(alwu)

According to [1], we're going to deprecate this attribute.

[1] https://github.com/w3c/media-playback-quality/pull/20

Attachment #9114992 - Attachment description: Bug 1602163 - part2 : remove usage of 'corruptedVideoFrames' in tests. → Bug 1602163 - part1 : remove usage of 'corruptedVideoFrames' in tests.
Attachment #9114990 - Attachment description: Bug 1602163 - part1 : remove 'corruptedFrames' from 'VideoPlaybackQuality' → Bug 1602163 - part2 : remove 'corruptedFrames' from 'VideoPlaybackQuality'
Pushed by alwu@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/8c086b8b46df
part1 : remove usage of 'corruptedVideoFrames' in tests. r=bzbarsky
https://hg.mozilla.org/integration/autoland/rev/a2bf666e1742
part2 : remove 'corruptedFrames' from 'VideoPlaybackQuality' r=bzbarsky
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/20714 for changes under testing/web-platform/tests
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla73
Upstream PR merged by jgraham
Flags: needinfo?(alwu)

Documentation updated:

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

Attachment

General

Created:
Updated:
Size: