Some tests around surface creation and snapshotting with color space

src/gpu/GrDrawingManager.cpp

 /*
  * Copyright 2015 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 #include "GrDrawContext.h"
 #include "GrDrawingManager.h"
 #include "GrDrawTarget.h"
@@ skipping 166 matching lines @@
     if (!pr && allowSW) {
         if (!fSoftwarePathRenderer) {
             fSoftwarePathRenderer = new GrSoftwarePathRenderer(fContext->textureProvider());
         }
         pr = fSoftwarePathRenderer;
     }
 
     return pr;
 }
 
+static bool checkConfigAndColorSpace(GrPixelConfig config, SkColorSpace* colorSpace) {

[Brian Osman, 2016/08/23 19:54:46]

I'm sad that I have this code in three places, now, but this version operates on
GrPixelConfig... sigh.

+    switch (config) {
+        case kRGBA_half_GrPixelConfig:
+            return colorSpace && SkColorSpace::kLinear_GammaNamed == colorSpace->gammaNamed();
+        case kSRGBA_8888_GrPixelConfig:
+        case kSBGRA_8888_GrPixelConfig:
+            return colorSpace && colorSpace->gammaCloseToSRGB();
+        default:
+            return !colorSpace;
+    }
+}
+
 sk_sp<GrDrawContext> GrDrawingManager::makeDrawContext(sk_sp<GrRenderTarget> rt,
                                                        sk_sp<SkColorSpace> colorSpace,
                                                        const SkSurfaceProps* surfaceProps) {
     if (this->wasAbandoned()) {
         return nullptr;
     }
 
+    if (!checkConfigAndColorSpace(rt->config(), colorSpace.get())) {
+        return nullptr;
+    }
 
     bool useDIF = false;
     if (surfaceProps) {
         useDIF = surfaceProps->isUseDeviceIndependentFonts();
     }
 
     if (useDIF && fContext->caps()->shaderCaps()->pathRenderingSupport() &&
         rt->isStencilBufferMultisampled()) {
         GrStencilAttachment* sb = fContext->resourceProvider()->attachStencilAttachment(rt.get());
         if (sb) {
             return sk_sp<GrDrawContext>(new GrPathRenderingDrawContext(
                                                         fContext, this, std::move(rt),
                                                         std::move(colorSpace), surfaceProps,
                                                         fContext->getAuditTrail(), fSingleOwner));
         }
     }
 
     return sk_sp<GrDrawContext>(new GrDrawContext(fContext, this, std::move(rt),
                                                   std::move(colorSpace), surfaceProps,
                                                   fContext->getAuditTrail(),
                                                   fSingleOwner));
 }