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 e7de3694df71eae5b1eedc2f4cc57d1f6ad542f7..a3178aec28825f792ca6718b4a4c7b0d4692dc17 100644
--- a/tests/ImageTest.cpp
+++ b/tests/ImageTest.cpp
@@ -914,15 +914,16 @@ DEF_GPUTEST_FOR_RENDERING_CONTEXTS(DeferredTextureImage, reporter, ctxInfo) {
           kNone_SkFilterQuality, 1, false },
         { [context] { return create_gpu_image(context); },
           {{SkMatrix::I(), kNone_SkFilterQuality, 0}},
-          kNone_SkFilterQuality, 1, false },
+          kNone_SkFilterQuality, 1, true },
         // Create a texture image in a another GrContext.
+        // ////////////////
         { [testContext, otherContextInfo] {
             otherContextInfo.testContext()->makeCurrent();
             sk_sp<SkImage> otherContextImage = create_gpu_image(otherContextInfo.grContext());

[cblume, 2016/10/25 02:51:01]

This test is now causing a errors on non-GPU:
../../tests/ImageTest.cpp:857   Expected image pixels to be the same. At 5,5
0xff000000 != 0xffffffff
I'm looking into it now.

[bsalomon, 2016/10/25 18:27:15]

I think this one should still be false, right? If we call
SkImage::getDeferredTextureImageData on an image that has a texture in Context A
with a proxy for Context B it seems simpler to fail rather than to try to do a
readback. This doesn't seem like a likely scenario for Chrome. If we are doing a
readback with the new code (not sure why that'd be) and it is failing then my
guess is that when we do the readback Context A isn't current so we're trying to
readback a texture ID from Context A in Context B.

             testContext->makeCurrent();
             return otherContextImage;
           }, {{SkMatrix::I(), kNone_SkFilterQuality, 0}},
-        kNone_SkFilterQuality, 1, false },
+        kNone_SkFilterQuality, 1, true },
         // Create an image that is too large to upload.
         { create_image_large,    {{SkMatrix::I(), kNone_SkFilterQuality, 0}},
           kNone_SkFilterQuality, 1, false },