Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(6)

Issue 1387313002: Add J444ToARGB conversion function. (Closed)

Created:
5 years, 2 months ago by fbarchard1
Modified:
5 years, 2 months ago
Reviewers:
harryjin
Base URL:
https://chromium.googlesource.com/libyuv/libyuv@master
Target Ref:
refs/heads/master
Project:
libyuv
Visibility:
Public.

Description

Add J444ToARGB conversion function. J444 is JPeg YUV color space with 444 subsampling. This implementation uses the existing I444ToARGB conversion, which is BT.601 color space with 444 subsampling, but passing in the jpeg color matrix constants. TBR=harryjin@google.com BUG=449 Committed: https://chromium.googlesource.com/libyuv/libyuv/+/f00bc9ef46d64b10e8e32cd38009a76c73a746b9

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+50 lines, -8 lines) Patch
M README.chromium View 1 chunk +1 line, -1 line 0 comments Download
M include/libyuv/convert_argb.h View 1 chunk +8 lines, -0 lines 0 comments Download
M include/libyuv/version.h View 1 chunk +1 line, -1 line 0 comments Download
M source/convert_argb.cc View 3 chunks +39 lines, -6 lines 0 comments Download
M unit_test/convert_test.cc View 1 chunk +1 line, -0 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
fbarchard1
5 years, 2 months ago (2015-10-07 01:46:28 UTC) #1
fbarchard1
Committed patchset #1 (id:1) manually as f00bc9ef46d64b10e8e32cd38009a76c73a746b9 (presubmit successful).
5 years, 2 months ago (2015-10-07 01:47:02 UTC) #2
harryjin
5 years, 2 months ago (2015-10-07 05:29:34 UTC) #3
Message was sent while issue was closed.
lgtm

Powered by Google App Engine
This is Rietveld 408576698