Add ability to query read pixels support without a render target.

src/gpu/gl/GrGLGpu.h

Index: src/gpu/gl/GrGLGpu.h
diff --git a/src/gpu/gl/GrGLGpu.h b/src/gpu/gl/GrGLGpu.h
index cde50193bfb8fd4376c5e1d7b6a90dd165e2b4e5..5dc7273b7762c8d0dc2f4a7e19ca203b8257a991 100644
--- a/src/gpu/gl/GrGLGpu.h
+++ b/src/gpu/gl/GrGLGpu.h
@@ -164,8 +164,20 @@ private:
     bool onMakeCopyForTextureParams(GrTexture*, const GrTextureParams&,
                                     GrTextureProducer::CopyParams*) const override;
 

[robertphillips, 2016/01/22 14:56:10]

pixels -> pixel ?
Here and in other two comments too

[bsalomon, 2016/01/22 15:50:19]

Done.

+    // Checks whether glReadPixels can be called to get pixels values in readConfig from the
+    // render target.
     bool readPixelsSupported(GrRenderTarget* target, GrPixelConfig readConfig);
 
+    // Checks whether glReadPixels can be called to get pixels values in readConfig from a
+    // render target that has renderTargetConfig. This may have to create a temporary
+    // render target and thus is less preferable than the variant that takes a render target.
+    bool readPixelsSupported(GrPixelConfig renderTargetConfig, GrPixelConfig readConfig);
+
+    // Checks whether glReadPixels can be called to get pixels values in readConfig from a
+    // render target that has the same config as surfaceForConfig. Calls one of the the two
+    // variations above, depending on whether the surface is a render target or not.
+    bool readPixelsSupported(GrSurface* surfaceForConfig, GrPixelConfig readConfig);
+
     bool onReadPixels(GrSurface*,
                       int left, int top,
                       int width, int height,