diff options
| author |  reed <reed@google.com> | 2016-08-01 11:24:14 -0700 |
|---|---|---|
| committer |  Commit bot <commit-bot@chromium.org> | 2016-08-01 11:24:14 -0700 |
| commit | f1ac18229c3f0f813a128a10111766c991f9dfcf (patch) | |
| tree | 0709fd5ca97bad6b19abfc7ecdd3fcd2190e8ef2 /src/pdf/SkPDFShader.cpp | |
| parent | da574d17f864ed70323a1c0fc6e4e969153a4c98 (diff) | |
implement isABitmap for imageshader, return localmatrix for bitmap's impl
For imageshader, I only return true if the image is explicitly raster-backed. I do not return true for texture, nor for generator (i.e. lazy/picture) backed. Is that ok?
BUG=skia:5592
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2197323002
Review-Url: https://codereview.chromium.org/2197323002
Diffstat (limited to 'src/pdf/SkPDFShader.cpp')
| -rw-r--r-- | src/pdf/SkPDFShader.cpp | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/src/pdf/SkPDFShader.cpp b/src/pdf/SkPDFShader.cpp index 31af569fe9..885b7e6049 100644 --- a/src/pdf/SkPDFShader.cpp +++ b/src/pdf/SkPDFShader.cpp @@ -1289,10 +1289,7 @@ SkPDFShader::State::State(SkShader* shader, const SkMatrix& canvasTransform, fType = shader->asAGradient(&fInfo); if (fType == SkShader::kNone_GradientType) { - SkMatrix matrix; - if (shader->isABitmap(&fImage, &matrix, fImageTileModes)) { - SkASSERT(matrix.isIdentity()); - } else { + if (!shader->isABitmap(&fImage, nullptr, fImageTileModes)) { // Generic fallback for unsupported shaders: // * allocate a bbox-sized bitmap // * shade the whole area |