From 6cc6ae7573fdfd0af1f9488349b1d2bb88c1b01d Mon Sep 17 00:00:00 2001 From: Matt Sarett Date: Tue, 18 Apr 2017 18:29:12 -0400 Subject: SkColorSpaceXformer: Add SkShader::makeColorSpace() Just a refactor - reimplements the shader xforms with makeColorSpace(). 11 gms have diffs. Some are down to floating precision. The old implementation would go float->fixed->float in some cases. Others are due to improvements with gradient shaders inside local matrix shaders. Bug: skia:6516 Change-Id: I424406990c5c58a47833cf4c9ef146cd3ea6c37e Reviewed-on: https://skia-review.googlesource.com/13769 Reviewed-by: Mike Klein Commit-Queue: Matt Sarett --- src/image/SkImageShader.h | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) (limited to 'src/image') diff --git a/src/image/SkImageShader.h b/src/image/SkImageShader.h index 8960812a06..5274826e88 100644 --- a/src/image/SkImageShader.h +++ b/src/image/SkImageShader.h @@ -8,9 +8,10 @@ #ifndef SkImageShader_DEFINED #define SkImageShader_DEFINED +#include "SkBitmapProcShader.h" +#include "SkColorSpaceXformer.h" #include "SkImage.h" #include "SkShader.h" -#include "SkBitmapProcShader.h" class SkImageShader : public SkShader { public: @@ -39,6 +40,11 @@ protected: bool onAppendStages(SkRasterPipeline*, SkColorSpace*, SkArenaAlloc*, const SkMatrix& ctm, const SkPaint&, const SkMatrix*) const override; + sk_sp onMakeColorSpace(SkColorSpaceXformer* xformer) const override { + return xformer->apply(fImage.get())->makeShader(fTileModeX, fTileModeY, + &this->getLocalMatrix()); + } + sk_sp fImage; const TileMode fTileModeX; const TileMode fTileModeY; -- cgit v1.2.3