AuroraMiddleware/skia2
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

SkImageCacherator: Remove unneeded heuristic

Browse Source 
This was the old way of handling color spaces.

This is now handled by chooseCacheFormat() / buildCacheInfo().

Bug: skia:
Change-Id: Ifab1ab25d547a00296af237578dd8befec9c86c6
Reviewed-on: https://skia-review.googlesource.com/13004
Reviewed-by: Brian Osman <brianosman@google.com>
Commit-Queue: Matt Sarett <msarett@google.com>
This commit is contained in:
Matt Sarett 2017-04-10 11:13:00 -04:00 committed by Skia Commit-Bot
parent 9f903e4a58
commit 891dde5b6f
1 changed files with 0 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
src/core/SkImageCacherator.cpp
View File

@ -94,17 +94,6 @@ SkImageCacherator::Validator::Validator(sk_sp<SharedGenerator> gen, const SkIRec
if (fInfo.colorType() == kIndex_8_SkColorType) {
fInfo = fInfo.makeColorType(kN32_SkColorType);
}
// If the encoded data is in a strange color space (it's not an XYZ matrix space), we won't be
// able to preserve the gamut of the encoded data when we decode it. Instead, we'll have to
// decode to a known color space (linear sRGB is a good choice). But we need to adjust the
// stored color space, because drawing code will ask the SkImage for its color space, which
// will in turn ask the cacherator. If we return the A2B color space, then we will be unable to
// construct a source-to-dest gamut transformation matrix.
if (fInfo.colorSpace() &&
SkColorSpace_Base::Type::kXYZ != as_CSB(fInfo.colorSpace())->type()) {
fInfo = fInfo.makeColorSpace(SkColorSpace::MakeSRGBLinear());
}
}
SkImageCacherator* SkImageCacherator::NewFromGenerator(std::unique_ptr<SkImageGenerator> gen,