commit | 97d4b16e5c57b45d462690321a901f28eda7995c | [log] [tgz] |
---|---|---|
author | Tommy Steimel <steimel@chromium.org> | Thu Jan 24 00:28:25 2019 |
committer | Commit Bot <commit-bot@chromium.org> | Thu Jan 24 00:28:25 2019 |
tree | 29c27b596ae0a367d2d88dbe60464785ec807262 | |
parent | b68b7b20e9ff79a6845e89215a093a9f5cf52c43 [diff] |
Revert "CC: Support YUV images in PaintImage and GpuImageDecodeCache" This reverts commit cb521877bf31b9b6df74d34e01da6e41b0d9b469. Reason for revert: causing compile errors on ios bots: https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/ios-device/112397 Original change's description: > CC: Support YUV images in PaintImage and GpuImageDecodeCache > > This patch modifies PaintImage to have an interface to indicate > that the underlying image is YUV pixel format and adds a > corresponding DecodeYuv() method. > > In GpuImageDecodeCache, this change adds "is YUV" variables/getters to > DecodedImageData, UploadedImageData (has_yuv_planes()), and ImageData, > as well as base:Optional arrays for the SkImages and GL texture IDs for > each Y, U, and V plane in the relevant places. > > The YUV decoding path will initially be implemented for in-process > GPU rasterization and later for OOPR (Out of Process Rasterization). > Changes that affect WebP decoding in Blink will be hidden behind a flag called > kDecodeLossyWebPImagesToYUV. > > Finally, because this decides whether to YUV decode based on > the result of PaintImageGenerator::QueryYUVA8, we separately modified > JPEGImageDecoder::CanDecodeToYUV (crrev.com/c/1400764) and > VideoImageGenerator::QueryYUVA8 (crrev.com/c/1418322) > to return false in order to avoid prematurely going down the YUV decode path. > > See bit.ly/webp-decoding-into-yuv for the design document and > crrev.com/c/1338461 for a working prototype CL. > > Bug: 900672 > Change-Id: I14f60d83cc2da2382e3da296fab598597b7e8ec8 > Reviewed-on: https://chromium-review.googlesource.com/c/1368687 > Reviewed-by: Miguel Casas <mcasas@chromium.org> > Reviewed-by: Khushal <khushalsagar@chromium.org> > Commit-Queue: Madeleine Barowsky <mbarowsky@chromium.org> > Cr-Commit-Position: refs/heads/master@{#625406} TBR=scroggo@chromium.org,ccameron@chromium.org,mcasas@chromium.org,dcastagna@chromium.org,khushalsagar@chromium.org,jvanverth@chromium.org,ericrk@chromium.org,andrescj@chromium.org,mbarowsky@chromium.org Change-Id: I5b67ca61424074b5d51c33692071a4306803c283 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 900672 Reviewed-on: https://chromium-review.googlesource.com/c/1432878 Reviewed-by: Tommy Steimel <steimel@chromium.org> Commit-Queue: Tommy Steimel <steimel@chromium.org> Cr-Commit-Position: refs/heads/master@{#625420}
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 .