commit f1ac18229c3f0f813a128a10111766c991f9dfcf
author reed <reed@google.com> Mon Aug 01 11:24:14 2016 -0700
committer Commit bot <commit-bot@chromium.org> Mon Aug 01 11:24:14 2016 -0700
tree 0709fd5ca97bad6b19abfc7ecdd3fcd2190e8ef2
parent da574d17f864ed70323a1c0fc6e4e969153a4c98

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

src/pdf/SkPDFShader.cpp

diff --git a/src/pdf/SkPDFShader.cpp b/src/pdf/SkPDFShader.cpp
index 31af569..885b7e6 100644
--- a/src/pdf/SkPDFShader.cpp
+++ b/src/pdf/SkPDFShader.cpp
@@ -1289,10 +1289,7 @@
     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