Fix setColocatedSampleLocations on ES and GL < 4.5

src/gpu/gl/GrGLInterface.cpp

Index: src/gpu/gl/GrGLInterface.cpp
diff --git a/src/gpu/gl/GrGLInterface.cpp b/src/gpu/gl/GrGLInterface.cpp
index 71de9521c2236d35d830d6a036f9c217b4a5effa..d47b629d17a35bbb07e3f8102ea0305e2c892a30 100644
--- a/src/gpu/gl/GrGLInterface.cpp
+++ b/src/gpu/gl/GrGLInterface.cpp
@@ -713,6 +713,15 @@ bool GrGLInterface::validate() const {
         }
     }
 
+    if ((kGL_GrGLStandard == fStandard && glVer >= GR_GL_VER(4,3)) ||
+        (kGLES_GrGLStandard == fStandard && glVer >= GR_GL_VER(3,1)) ||
+        fExtensions.has("GL_NV_sample_locations") ||

[bsalomon, 2015/11/03 15:21:12]

Here you check for the extensions but not in the interface assembler code?

[Chris Dalton, 2015/11/03 15:49:15]

Yeah this check was a little OCD. glFramebufferParameteri is core, but the idea
was that these extensions depend on it (they're written against OpenGL versions
that have this function). I can just take the check out for the extensions.

[bsalomon, 2015/11/03 15:54:22]

I suppose if a vendor shipped an ES 3.0 with the extension advertised then we'd
expect to be able to get the function. So maybe update the assembler code? I
don't really have a strong opinion though, so happy to go with whatever you
think is best.

+        fExtensions.has("GL_ARB_sample_locations")) {
+        if (nullptr == fFunctions.fFramebufferParameteri) {
+            RETURN_FALSE_INTERFACE
+        }
+    }
+
     if (kGL_GrGLStandard == fStandard && glVer >= GR_GL_VER(4,5)) {
         if (nullptr == fFunctions.fNamedFramebufferParameteri) {
             RETURN_FALSE_INTERFACE