commit 677bb6ffc06e2ab3eeacead5ad98861dad8bf35a
author Geoff Lang <geofflang@chromium.org> Wed Apr 05 12:40:40 2017 -0400
committer Commit Bot <commit-bot@chromium.org> Thu May 04 17:05:10 2017 +0000
tree 2fac8f22c6f8ab1016cbff19a3bd982e47ea2e8a
parent bb7e5a7c08c60cab7e80242ab03d5cc0e415933b

Update checks for floating point renderability.

 * Expose GL_CHROMIUM_color_buffer_float_rgb and
   GL_CHROMIUM_color_buffer_float_rgba
 * Fix many texture formats that were incorrectly checking the wrong
   extension for support or renderability.
 * Make all floating point texture extensions dynamically enableable.

BUG=angleproject:1958
BUG=angleproject:1715

Change-Id: Iefccc8b5ae5edd97623affa9de05b1d9af5c9598
Reviewed-on: https://chromium-review.googlesource.com/468450
Reviewed-by: Jamie Madill <jmadill@chromium.org>
Commit-Queue: Geoff Lang <geofflang@chromium.org>

src/libANGLE/validationES2.cpp

diff --git a/src/libANGLE/validationES2.cpp b/src/libANGLE/validationES2.cpp
index 417502a..b4fdb1e 100644
--- a/src/libANGLE/validationES2.cpp
+++ b/src/libANGLE/validationES2.cpp
@@ -1290,11 +1290,23 @@
             break;
         case GL_R8_EXT:
         case GL_RG8_EXT:
+            if (!context->getExtensions().textureRG)
+            {
+                context->handleError(Error(GL_INVALID_ENUM));
+                return false;
+            }
+            break;
         case GL_R16F_EXT:
         case GL_RG16F_EXT:
+            if (!context->getExtensions().textureRG || !context->getExtensions().textureHalfFloat)
+            {
+                context->handleError(Error(GL_INVALID_ENUM));
+                return false;
+            }
+            break;
         case GL_R32F_EXT:
         case GL_RG32F_EXT:
-            if (!context->getExtensions().textureRG)
+            if (!context->getExtensions().textureRG || !context->getExtensions().textureFloat)
             {
                 context->handleError(Error(GL_INVALID_ENUM));
                 return false;