commit c14ab2a7aaf244ec6b96b61f91d5251accf54605
author Shahbaz Youssefi <syoussefi@google.com> Fri Aug 24 13:57:55 2018 -0400
committer Commit Bot <commit-bot@chromium.org> Mon Aug 27 15:39:56 2018 +0000
tree 71f2da415d115dc1d4388a6f9b569abdc933f982
parent 89664843c656a97b82f8c71781bf8d451f7f8ec3

Fix dEQP_GLES3.functional.negative_api.shader.program_binary

The frontend was not taking into account the fact that some drivers do
not support program binaries.  The fix is that now

    glGetProgramiv(GL_PROGRAM_BINARY_LENGTH)

returns 0 under such circumstances.

Bug: angleproject:2780

Change-Id: I4ea2cda6bb4612e69dbd7e00b3d43b97ee3b3c3d
Reviewed-on: https://chromium-review.googlesource.com/1188768
Reviewed-by: Geoff Lang <geofflang@chromium.org>
Reviewed-by: Jamie Madill <jmadill@chromium.org>
Commit-Queue: Shahbaz Youssefi <syoussefi@chromium.org>

src/libANGLE/queryutils.cpp

diff --git a/src/libANGLE/queryutils.cpp b/src/libANGLE/queryutils.cpp
index c16fefa..8c7e838 100644
--- a/src/libANGLE/queryutils.cpp
+++ b/src/libANGLE/queryutils.cpp
@@ -1043,7 +1043,9 @@
             *params = program->getActiveUniformMaxLength();
             return;
         case GL_PROGRAM_BINARY_LENGTH_OES:
-            *params = program->getBinaryLength(context);
+            *params = context->getCaps().programBinaryFormats.empty()
+                          ? 0
+                          : program->getBinaryLength(context);
             return;
         case GL_ACTIVE_UNIFORM_BLOCKS:
             *params = program->getActiveUniformBlockCount();