Added --deepColor option to SampleApp, triggers creation of a ten-bit/channel buffer on Windows.

include/gpu/GrContext.h

Index: include/gpu/GrContext.h
diff --git a/include/gpu/GrContext.h b/include/gpu/GrContext.h
index cb407aa5bfa16db1d2c3e7800e0c2c237e7b2804..9d0e4b7c81a4e61a0a8ca3dab8ee960aab11f86a 100644
--- a/include/gpu/GrContext.h
+++ b/include/gpu/GrContext.h
@@ -298,6 +298,11 @@ public:
                             uint32_t pixelOpsFlags = 0);
 
     /**
+     * Writes the texture to the render target, while applying a gamma curve.
+     */

[bsalomon, 2016/04/29 18:27:20]

Question about this API:

Can dst and src be different sizes? If so what happens? Should it take a
srcRect, dstPoint a la copySurface?

Perhaps we should have some unit tests for this (CopySurfaceTest.cpp might be a
decent starting point).

[Brian Osman, 2016/05/02 21:02:45]

I started trying to support different sizes, but the implementation isn't quite
the same as copySurface (due to different back-end code paths). I can imagine
that people may need that functionality eventually, but I'd rather wait until
it's required. I'm updating the documentation (and code) to make it clear that
it's required for them to be identically sized.

I will add some unit tests, although with the size restriction, the test matrix
will obviously be much smaller.

[bsalomon, 2016/05/02 21:17:40]

sgtm

+    bool applyGamma(GrRenderTarget* dst, GrTexture* src, SkScalar gamma);
+
+    /**
      * Copies a rectangle of texels from src to dst.
      * bounds.
      * @param dst           the surface to copy to.