Don't copy image buffer when calculating size.

src/image/SkImage_Gpu.cpp

Index: src/image/SkImage_Gpu.cpp
diff --git a/src/image/SkImage_Gpu.cpp b/src/image/SkImage_Gpu.cpp
index 06ec224a2a9cbc23fa02b83288128a4252329b26..59db33db91870f599f65cba9e050d242080fe7b4 100644
--- a/src/image/SkImage_Gpu.cpp
+++ b/src/image/SkImage_Gpu.cpp
@@ -488,7 +488,7 @@ size_t SkImage::getDeferredTextureImageData(const GrContextThreadSafeProxy& prox
         // Here we're just using presence of data to know whether there is a codec behind the image.
         // In the future we will access the cacherator and get the exact data that we want to (e.g.

[vmpstr, 2016/09/12 23:35:27]

nit: You should update the comment.

[cblume, 2016/09/13 00:02:26]

Done.

         // yuv planes) upload.
-        sk_sp<SkData> data(this->refEncoded());
+        SkImageCacherator* data(as_IB(this)->peekCacherator());

[vmpstr, 2016/09/12 23:35:27]

Can we just inline this in the if? I don't think we use data anywhere else.

[cblume, 2016/09/13 00:02:26]

Done. This actually makes it read much more easily to me as well:
if(!peekCache() && !peekPixels()) essentially.

         if (!data && !this->peekPixels(nullptr)) {
             return 0;
         }