commit | db2bf67573cc0ff295f995e5746eb7516df0d26e | [log] [tgz] |
---|---|---|
author | Hirokazu Honda <hiroh@chromium.org> | Tue Nov 20 00:13:16 2018 |
committer | Commit Bot <commit-bot@chromium.org> | Tue Nov 20 00:13:16 2018 |
tree | a6cd0135bbd61a9e262ca7c192dd9d94911667dc | |
parent | c1b5b2a2b60e43a8c9565a8f4c5d1b98eb837f4c [diff] |
GenericDmaBufVideoFrameValidator: Map YV12 VideoFrame as-is GenericDmaBufVideoFrameValidator swaps U plane and V plane addresses so that VideoFrameValidator is able to libyuv::I420Copy in the order Y, U and V. This is a wrong design, meaning VideoFrameMapper does not only map but also does something. VideoFrameMapper caller should expect the address acquired by VideoFrame::data(UPlane) is V plane's and one by VideoFrame::data(VPlane) is U plane's in the case of YV12 VideoFrame. BUG=chromium:895230, chromium:856562 TEST=VDA unittest on hana with --frame_validator=check TEST=VEA unittest on hana with --native_input Change-Id: Ibf5e5685d95634fe6720f07dc154fd01907516d8 Reviewed-on: https://chromium-review.googlesource.com/c/1341783 Reviewed-by: Pawel Osciak <posciak@chromium.org> Commit-Queue: Hirokazu Honda <hiroh@chromium.org> Cr-Commit-Position: refs/heads/master@{#609518}
Chromium is an open-source browser project that aims to build a safer, faster, and more stable way for all users to experience the web.
The project's web site is https://www.chromium.org.
Documentation in the source is rooted in docs/README.md.
Learn how to Get Around the Chromium Source Code Directory Structure .