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

Issues Search
    My Issues | Starred     Open | Closed | All

Issue 2146733002: Test nv21 layout of Android420ToI420 function. (Closed)

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

Description

Test nv21 layout of Android420ToI420 function. to Y,U,V and a pixel stride for U and V. The pixel stride is expected to be 1 or 2. [ RUN ] LibYUVConvertTest.Android420ToI420_1_Any [ OK ] LibYUVConvertTest.Android420ToI420_1_Any (253 ms) [ RUN ] LibYUVConvertTest.Android420ToI420_1_Unaligned [ OK ] LibYUVConvertTest.Android420ToI420_1_Unaligned (250 ms) [ RUN ] LibYUVConvertTest.Android420ToI420_1_Invert [ OK ] LibYUVConvertTest.Android420ToI420_1_Invert (254 ms) [ RUN ] LibYUVConvertTest.Android420ToI420_1_Opt [ OK ] LibYUVConvertTest.Android420ToI420_1_Opt (247 ms) [ RUN ] LibYUVConvertTest.Android420ToI420_2_Any [ OK ] LibYUVConvertTest.Android420ToI420_2_Any (132 ms) [ RUN ] LibYUVConvertTest.Android420ToI420_2_Unaligned [ OK ] LibYUVConvertTest.Android420ToI420_2_Unaligned (122 ms) [ RUN ] LibYUVConvertTest.Android420ToI420_2_Invert [ OK ] LibYUVConvertTest.Android420ToI420_2_Invert (124 ms) [ RUN ] LibYUVConvertTest.Android420ToI420_2_Opt [ OK ] LibYUVConvertTest.Android420ToI420_2_Opt (119 ms) TEST=LibYUVConvertTest.Android420ToI420_Opt BUG=libyuv:604 R=braveyao@chromium.org Committed: https://chromium.googlesource.com/libyuv/libyuv/+/abcb70f183e90eff6c6e36ec97c48990205310bd

Patch Set 1 #

Patch Set 2 : bump version #

Patch Set 3 : test NV21 #

Patch Set 4 : lint space fixes #

Patch Set 5 : yuv3 test #

Unified diffs Side-by-side diffs Delta from patch set Stats (+38 lines, -37 lines) Patch
M README.chromium View 1 1 chunk +1 line, -1 line 0 comments Download
M include/libyuv/version.h View 1 1 chunk +1 line, -1 line 0 comments Download
M source/convert.cc View 1 1 chunk +1 line, -1 line 0 comments Download
M unit_test/convert_test.cc View 1 2 3 4 5 chunks +35 lines, -34 lines 0 comments Download

Messages

Total messages: 5 (1 generated)
fbarchard1
4 years, 5 months ago (2016-07-13 00:37:52 UTC) #1
fbarchard1
Tests with '1' in name are pixel stride of 1, which is I420 Tests with ...
4 years, 5 months ago (2016-07-13 00:48:55 UTC) #2
braveyao
lgtm
4 years, 5 months ago (2016-07-13 00:54:20 UTC) #3
fbarchard1
4 years, 5 months ago (2016-07-13 01:34:09 UTC) #5
Message was sent while issue was closed. 
Committed patchset #5 (id:80001) manually as
abcb70f183e90eff6c6e36ec97c48990205310bd (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698