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 0acb428d7a3bc85cbd9a2f2d802d97a4fd842e78..149ead43ed1f65590097947cac1a9f7010639c53 100644
--- a/tests/ImageTest.cpp
+++ b/tests/ImageTest.cpp
@@ -920,7 +920,7 @@ DEF_GPUTEST_FOR_RENDERING_CONTEXTS(DeferredTextureImage, reporter, ctxInfo) {
             testContext->makeCurrent();
             return otherContextImage;
           }, {{SkMatrix::I(), kNone_SkFilterQuality, 0}},
-        kNone_SkFilterQuality, 1, false },
+        kNone_SkFilterQuality, 1, true },

[cblume, 2016/11/16 01:07:16]

The true/false here is a bit unclear.

If it is cheap to create a texture (and thus, we don't need to allocate memory
to create one), we return 0 size required to be allocated.
0 == false.

So the scenario where it is cheap to create means an expected outcome of false.
It doesn't really mean the test failed.


This particular line is for multiple contexts, where creating a texture is not
cheap.

         // Create an image that is too large to upload.
         { create_image_large,    {{SkMatrix::I(), kNone_SkFilterQuality, 0}},
           kNone_SkFilterQuality, 1, false },