Enable stencil clipping in mixed sampled render targets

src/gpu/gl/GrGLInterface.cpp

 /*
  * Copyright 2011 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 
 #include "gl/GrGLInterface.h"
 #include "gl/GrGLExtensions.h"
@@ skipping 683 matching lines @@
                 nullptr == fFunctions.fGetVertexArrayPointerv ||
                 nullptr == fFunctions.fGetVertexArrayIntegeri_v ||
                 nullptr == fFunctions.fGetVertexArrayPointeri_v ||
                 nullptr == fFunctions.fMapNamedBufferRange ||
                 nullptr == fFunctions.fFlushMappedNamedBufferRange) {
                 RETURN_FALSE_INTERFACE
             }
         }
     }
 
+    if (glVer >= GR_GL_VER(4,3)) {

[Chris Dalton, 2015/09/22 08:35:24]

I think we also need to check the GL standard and make sure it isn't GLES before
validating FramebufferParameteri.

+        if (NULL == fFunctions.fFramebufferParameteri) {
+            RETURN_FALSE_INTERFACE
+        }
+        if (NULL == fFunctions.fNamedFramebufferSampleLocationsfv) {

[Chris Dalton, 2015/09/22 08:35:24]

We should validate for this method when GL_NV_sample_locations is present,
correct? (Not GL version 4.3?)

+            RETURN_FALSE_INTERFACE
+        }
+        if (NULL == fFunctions.fGetMultisamplefv) {

[Chris Dalton, 2015/09/22 08:35:24]

We can validate for this method on GL 3.2 and GLES 3.1.

+            RETURN_FALSE_INTERFACE
+        }
+    }
+
     if ((kGL_GrGLStandard == fStandard && glVer >= GR_GL_VER(4,3)) ||
         fExtensions.has("GL_KHR_debug")) {
         if (nullptr == fFunctions.fDebugMessageControl ||
             nullptr == fFunctions.fDebugMessageInsert ||
             nullptr == fFunctions.fDebugMessageCallback ||
             nullptr == fFunctions.fGetDebugMessageLog ||
             nullptr == fFunctions.fPushDebugGroup ||
             nullptr == fFunctions.fPopDebugGroup ||
             nullptr == fFunctions.fObjectLabel) {
             RETURN_FALSE_INTERFACE
         }
     }
 
     return true;
 }