diff options
author | 2016-08-18 14:50:56 -0700 | |
---|---|---|
committer | 2016-08-18 14:50:56 -0700 | |
commit | da082a5767d7edfd3abe74fc683392422565a606 (patch) | |
tree | 9e4d6618abcf040a7f599b514e43b7bf98b3f8fe /src/effects | |
parent | 0367568d4af6a8ffd75bfd07bcb0172fef415940 (diff) |
Return color shader instead of 2-color gradient when color count is 1
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2259823005
Review-Url: https://codereview.chromium.org/2259823005
Diffstat (limited to 'src/effects')
-rw-r--r-- | src/effects/gradients/SkGradientShader.cpp | 29 |
1 files changed, 12 insertions, 17 deletions
diff --git a/src/effects/gradients/SkGradientShader.cpp b/src/effects/gradients/SkGradientShader.cpp index 6ffe56b9d4..3b761a51ca 100644 --- a/src/effects/gradients/SkGradientShader.cpp +++ b/src/effects/gradients/SkGradientShader.cpp @@ -742,18 +742,6 @@ static bool valid_grad(const SkColor colors[], const SkScalar pos[], int count, return nullptr != colors && count >= 1 && tileMode < (unsigned)SkShader::kTileModeCount; } -// assumes colors is SkColor* and pos is SkScalar* -#define EXPAND_1_COLOR(count) \ - SkColor tmp[2]; \ - do { \ - if (1 == count) { \ - tmp[0] = tmp[1] = colors[0]; \ - colors = tmp; \ - pos = nullptr; \ - count = 2; \ - } \ - } while (0) - static void desc_init(SkGradientShaderBase::Descriptor* desc, const SkColor colors[], const SkScalar pos[], int colorCount, SkShader::TileMode mode, uint32_t flags, const SkMatrix* localMatrix) { @@ -779,7 +767,9 @@ sk_sp<SkShader> SkGradientShader::MakeLinear(const SkPoint pts[2], if (!valid_grad(colors, pos, colorCount, mode)) { return nullptr; } - EXPAND_1_COLOR(colorCount); + if (1 == colorCount) { + return SkShader::MakeColorShader(colors[0]); + } SkGradientShaderBase::Descriptor desc; desc_init(&desc, colors, pos, colorCount, mode, flags, localMatrix); @@ -798,7 +788,9 @@ sk_sp<SkShader> SkGradientShader::MakeRadial(const SkPoint& center, SkScalar rad if (!valid_grad(colors, pos, colorCount, mode)) { return nullptr; } - EXPAND_1_COLOR(colorCount); + if (1 == colorCount) { + return SkShader::MakeColorShader(colors[0]); + } SkGradientShaderBase::Descriptor desc; desc_init(&desc, colors, pos, colorCount, mode, flags, localMatrix); @@ -826,8 +818,9 @@ sk_sp<SkShader> SkGradientShader::MakeTwoPointConical(const SkPoint& start, return SkShader::MakeEmptyShader(); } } - - EXPAND_1_COLOR(colorCount); + if (1 == colorCount) { + return SkShader::MakeColorShader(colors[0]); + } bool flipGradient = startRadius > endRadius; @@ -867,7 +860,9 @@ sk_sp<SkShader> SkGradientShader::MakeSweep(SkScalar cx, SkScalar cy, if (!valid_grad(colors, pos, colorCount, SkShader::kClamp_TileMode)) { return nullptr; } - EXPAND_1_COLOR(colorCount); + if (1 == colorCount) { + return SkShader::MakeColorShader(colors[0]); + } SkGradientShaderBase::Descriptor desc; desc_init(&desc, colors, pos, colorCount, SkShader::kClamp_TileMode, flags, localMatrix); |