Closed Bug 1735893 Opened 1 year ago Closed 1 year ago

Restore code to allow uploading to IOSurfaces in NV12 format

Categories

(Core :: Graphics: Layers, task)

Product:
Core
Component:
Graphics: Layers
Type:
task

Tracking

()

Status:
RESOLVED FIXED
Milestone:
95 Branch
Tracking Flags:
Tracking Status
firefox95 --- fixed

People

(Reporter: mattwoodrow, Assigned: mattwoodrow)

Details

Attachments

(1 file)

Bug 1735893 - Add pref to use NV12 when uploading software-decoded video to an IOSurface. r?bradwerth
48 bytes, text/x-phabricator-request
Details | Review

We initially used NV12 format for uploading software decoded video on MacOS, but switched to the Mac-specific format 'yuvs' after some users had playback bugs (see bug 1657107).

It appears that CoreAnimation doesn't support yuvs for detached mode though, but it does for NV12.

This bug adds back the code for the NV12 path (behind a pref) so that we can test it more easily.

We'll still need to figure out the rendering issues (or blocklist affected users) before shipping it enabled.

Pushed by mwoodrow@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/5b05c21bd919
Add pref to use NV12 when uploading software-decoded video to an IOSurface. r=bradwerth

https://hg.mozilla.org/mozilla-central/rev/5b05c21bd919

Status: NEW → RESOLVED
Closed: 1 year ago
status-firefox95: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 95 Branch
You need to log in before you can comment on or make changes to this bug.