Don't copy image buffer when calculating size.

tests/ImageTest.cpp

Index: tests/ImageTest.cpp
diff --git a/tests/ImageTest.cpp b/tests/ImageTest.cpp
index e53d14222c7273a66c50afd512c6a7a2760fb21c..256dc5fceab4bfd77d8dec01263614d079de1c28 100644
--- a/tests/ImageTest.cpp
+++ b/tests/ImageTest.cpp
@@ -895,7 +895,7 @@ DEF_GPUTEST_FOR_RENDERING_CONTEXTS(DeferredTextureImage, reporter, ctxInfo) {
         { create_data_image,     {{SkMatrix::I(), kNone_SkFilterQuality, 0}},
           kNone_SkFilterQuality, 1, true },
         { create_picture_image,  {{SkMatrix::I(), kNone_SkFilterQuality, 0}},
-          kNone_SkFilterQuality, 1, false },
+          kNone_SkFilterQuality, 1, true },

[cblume, 2016/09/16 08:10:45]

I am not yet familiar with the cacherator but as I stepped through this code,
everything seemed to make sense.

Do we really expect this to be failing?

[bsalomon, 2016/09/16 13:23:49]

We definitely don't want this one to succeed. It will sw-rasterize a picture
during decode time instead of gpu-rasterizing it during normal rasterization.

Perhaps we should add a query to SkImage_Base or the cacherator about whether
the image is encoded (if there isn't one already).

         { [context] { return create_gpu_image(context); },
           {{SkMatrix::I(), kNone_SkFilterQuality, 0}},
           kNone_SkFilterQuality, 1, false },