diff options
| author |  Matt Sarett <msarett@google.com> | 2017-04-18 18:29:12 -0400 |
|---|---|---|
| committer |  Skia Commit-Bot <skia-commit-bot@chromium.org> | 2017-04-19 16:28:44 +0000 |
| commit | 6cc6ae7573fdfd0af1f9488349b1d2bb88c1b01d (patch) | |
| tree | 82902aa9d97585692885c62f144d883ecc035c74 /src/image | |
| parent | 4dbc9cead90eb31756a1c1bf61ea01f223c5fe96 (diff) | |
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 <mtklein@chromium.org>
Commit-Queue: Matt Sarett <msarett@google.com>
Diffstat (limited to 'src/image')
| -rw-r--r-- | src/image/SkImageShader.h | 8 |
1 files changed, 7 insertions, 1 deletions
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<SkShader> onMakeColorSpace(SkColorSpaceXformer* xformer) const override { + return xformer->apply(fImage.get())->makeShader(fTileModeX, fTileModeY, + &this->getLocalMatrix()); + } + sk_sp<SkImage> fImage; const TileMode fTileModeX; const TileMode fTileModeY; |