Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add _420ToARGB() and _420ToABGR with BT.709 color space support for SkCanvasVideoRenderer. #472

Open
GoogleCodeExporter opened this issue Sep 1, 2015 · 2 comments

Comments

@GoogleCodeExporter
Copy link

Required for this function:
https://code.google.com/p/chromium/codesearch#chromium/src/media/blink/skcanvas_
video_renderer.cc&l=543

BT709 constants available here:
https://code.google.com/p/chromium/codesearch#chromium/src/media/base/yuv_conver
t.cc&l=288

Original issue reported on code.google.com by [email protected] on 24 Jul 2015 at 9:44

@GoogleCodeExporter
Copy link
Author

Theres an SkCanvasVideoRenderer unittest in media_blink_unittests

Original comment by [email protected] on 24 Jul 2015 at 10:00

@GoogleCodeExporter
Copy link
Author

Code search only finds bt709 using the ConvertYUVToRGB32 function now

https://code.google.com/p/chromium/codesearch#search/&q=ConvertYUVToRGB32&sq=pac
kage:chromium&type=cs

Plan to make I420 conversion accept a color matrix.

Original comment by [email protected] on 28 Aug 2015 at 5:45

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant