Closed Bug 1649532 Opened 4 years ago Closed 4 years ago

Port remaining decoder doctor related messages to Fission

Categories

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

Desktop
Unspecified
enhancement

Tracking

()

RESOLVED FIXED
82 Branch
Fission Milestone M7
Tracking Status
firefox82 --- fixed

People

(Reporter: bryce, Assigned: bryce)

References

Details

Attachments

(2 files)

What is the user-facing effect of this? We need this to triage for Fission priority.

Fission Milestone: --- → ?
Flags: needinfo?(bvandyk)

(In reply to Neha Kochar [:neha] from comment #1)

What is the user-facing effect of this? We need this to triage for Fission priority.

Diagnostic information relating to media handling failures in gecko. I do not know the full extent and hope to get a better idea once I've picked this task up. As an example, one of the things I believe this does is show a notification for the case where we believe users do not have Windows codecs installed. That notification helps guide users towards installing the appropriate codec pack so we can then use Windows' decoding for proprietary codecs.

Flags: needinfo?(bvandyk)

Tracking for Fission M7 Beta. We should fix these Windows codec notifications before we ship Fission, but they probably don't need to block enabling Fission in Nightly (M6). Most Nightly users probably keep their systems up to date and have the necessary codecs installed.

Fission Milestone: ? → M7

Port handling of DecoderDoctor notifications for fission. This is done by moving
the observing of 'decoder-doctor-notification' from browser-media.js to fission
friendly actors.

As browser-media.js had only code related to this patch left it is removed along
with references to it.

Expand the decoder doctor browser chrome test to check that notifications
trigger the expected UI behaviour when sent from cross origin iframes.

Pushed by bvandyk@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/06da457da734 Port DecoderDoctor notifications to fission. r=Gijs https://hg.mozilla.org/integration/autoland/rev/4bb25142d23c Add cross origin DecoderDoctor notification test. r=Gijs
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 82 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: