Closed Bug 1423795 Opened 7 years ago Closed 7 years ago

Set correct YUVColorSpace for video if possible

Categories

(Core :: Graphics: WebRender, enhancement, P1)

Product:
Core
Component:
Graphics: WebRender
Type:
enhancement

Tracking

()

Status:
VERIFIED FIXED
Milestone:
mozilla59
Tracking Flags:
Tracking Status
firefox59 --- fixed

People

(Reporter: sotaro, Assigned: sotaro)

References

Details

(Whiteboard: [wr-mvp])

Attachments

(1 file, 1 obsolete file)

patch - Set correct YUVColorSpace for video if possible
5.64 KB, patch
sotaro
: review+
Details | Diff | Splinter Review 
Current WebRender always render video with BT601. It is not good. If we could get YUVColorSpace value, we should use it to render videos correctly.

Gecko does not get YUVColorSpace of videos in some cases like NV12 on windows:(
Assignee: nobody → sotaro.ikeda.g
Attachment #8935222 - Flags: review?(nical.bugzilla)
Attachment #8935222 - Flags: review?(nical.bugzilla) → review+
Status: NEW → ASSIGNED
Priority: -- → P1
Whiteboard: [wr-mvp]
Fixed build failure on linux.
Attachment #8935222 - Attachment is obsolete: true
Attachment #8935591 - Flags: review+
Pushed by sikeda@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/76e2381ca4d1
Set correct YUVColorSpace for video if possible r=nical
https://hg.mozilla.org/mozilla-central/rev/76e2381ca4d1
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
status-firefox59: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla59
Thanks, it works! :)
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: