Fix validation of querying GL_FRAGMENT_SHADER_DERIVATIVE_HINT.

This enum is queryable in ES3 or with the GL_OES_standard_derivatives extension. BUG=2110 Change-Id: I688777bcc65468d796333a24223d24cc66170788 Reviewed-on: https://chromium-review.googlesource.com/576156Reviewed-by: Corentin Wallez <cwallez@chromium.org> Commit-Queue: Geoff Lang <geofflang@chromium.org>

Fix validation of querying GL_FRAGMENT_SHADER_DERIVATIVE_HINT.
6170bef8 · Geoff Lang · Commit Bot · 9c335865 · 6170bef8
Commit 6170bef8 authored Jul 18, 2017 by Geoff Lang Committed by Commit Bot Jul 18, 2017
Hide whitespace changes
Inline Side-by-side

Showing with 9 additions and 1 deletion

ContextState.cpp src/libANGLE/ContextState.cpp +9 -1

No files found.
--- a/src/libANGLE/ContextState.cpp
+++ b/src/libANGLE/ContextState.cpp
@@ -181,7 +181,6 @@ bool ValidationContext::getQueryParameterInfo(GLenum pname, GLenum *type, unsign
        case GL_PACK_REVERSE_ROW_ORDER_ANGLE:
        case GL_UNPACK_ALIGNMENT:
        case GL_GENERATE_MIPMAP_HINT:
-        case GL_FRAGMENT_SHADER_DERIVATIVE_HINT_OES:
        case GL_RED_BITS:
        case GL_GREEN_BITS:
        case GL_BLUE_BITS:
@@ -492,6 +491,15 @@ bool ValidationContext::getQueryParameterInfo(GLenum pname, GLenum *type, unsign
            *type      = GL_INT;
            *numParams = 1;
            return true;
+
+            case GL_FRAGMENT_SHADER_DERIVATIVE_HINT:
+                if ((getClientMajorVersion() < 3) && !getExtensions().standardDerivatives)
+                {
+                    return false;
+                }
+                *type      = GL_INT;
+                *numParams = 1;
+                return true;
        }
    }