commit 35d1501b7bcbacd7af96a65a52fb545c772e41bf
author Jamie Madill <jmadill@chromium.org> Mon Oct 07 10:46:37 2013 -0400
committer Jamie Madill <jmadill@chromium.org> Tue Oct 08 14:27:01 2013 -0400
tree f23d077c443c78e7fe0d7ee37e27394bd9a56b60
parent 33f9c2ef353f4156b975387102252873bab11350

Clean up the validation logic in glGenerateMipmap by using the generic base format property methods.

TRAC #23959

Signed-off-by: Shannon Woods
Signed-off-by: Geoff Lang

src/libGLESv2/libGLESv2.cpp

diff --git a/src/libGLESv2/libGLESv2.cpp b/src/libGLESv2/libGLESv2.cpp
index eddbab4..fff0994 100644
--- a/src/libGLESv2/libGLESv2.cpp
+++ b/src/libGLESv2/libGLESv2.cpp
@@ -27,15 +27,11 @@
 #include "libGLESv2/validationES2.h"
 #include "libGLESv2/validationES3.h"
 
-
-gl::Texture *getTargetTexture(gl::Context *context, GLenum target)
+gl::Texture *GetTargetTexture(gl::Context *context, GLenum target)
 {
-    if (context->getClientVersion() < 3)
+    if (!ValidTextureTarget(context, target))
     {
-        if (target == GL_TEXTURE_3D || target == GL_TEXTURE_2D_ARRAY)
-        {
-            return NULL;
-        }
+        return NULL;
     }
 
     switch (target)
@@ -48,7 +44,6 @@
     }
 }
 
-
 extern "C"
 {
 
@@ -2204,74 +2199,20 @@
 
         if (context)
         {
-            gl::Texture *texture = NULL;
-            GLint internalFormat = GL_NONE;
-
-            switch (target)
+            if (!ValidTextureTarget(context, target))
             {
-              case GL_TEXTURE_2D:
-                {
-                    gl::Texture2D *tex2d = context->getTexture2D();
-                    if (tex2d)
-                    {
-                        internalFormat = tex2d->getInternalFormat(0);
-                        texture = tex2d;
-                    }
-                    break;
-                }
-
-              case GL_TEXTURE_CUBE_MAP:
-                {
-                    gl::TextureCubeMap *texcube = context->getTextureCubeMap();
-                    if (texcube)
-                    {
-                        internalFormat = texcube->getInternalFormat(GL_TEXTURE_CUBE_MAP_POSITIVE_X, 0);
-                        texture = texcube;
-                    }
-                    break;
-                }
-
-              case GL_TEXTURE_3D:
-                {
-                    if (context->getClientVersion() < 3)
-                    {
-                        return gl::error(GL_INVALID_ENUM);
-                    }
-
-                    gl::Texture3D *tex3D = context->getTexture3D();
-                    if (tex3D)
-                    {
-                        internalFormat = tex3D->getInternalFormat(0);
-                        texture = tex3D;
-                    }
-                    break;
-                }
-
-                case GL_TEXTURE_2D_ARRAY:
-                  {
-                      if (context->getClientVersion() < 3)
-                      {
-                          return gl::error(GL_INVALID_ENUM);
-                      }
-
-                      gl::Texture2DArray *tex2darr = context->getTexture2DArray();
-                      if (tex2darr)
-                      {
-                          internalFormat = tex2darr->getInternalFormat(0);
-                          texture = tex2darr;
-                      }
-                      break;
-                  }
-
-              default:
                 return gl::error(GL_INVALID_ENUM);
             }
 
-            if (!texture)
+            gl::Texture *texture = GetTargetTexture(context, target);
+
+            if (texture == NULL)
             {
                 return gl::error(GL_INVALID_OPERATION);
             }
 
+            GLint internalFormat = texture->getBaseLevelInternalFormat();
+
             // Internally, all texture formats are sized so checking if the format
             // is color renderable and filterable will not fail.
             if (gl::IsDepthRenderingSupported(internalFormat, context) ||
@@ -3744,7 +3685,7 @@
 
         if (context)
         {
-            gl::Texture *texture = getTargetTexture(context, target);
+            gl::Texture *texture = GetTargetTexture(context, target);
 
             if (!texture)
             {
@@ -3814,7 +3755,7 @@
 
         if (context)
         {
-            gl::Texture *texture = getTargetTexture(context, target);
+            gl::Texture *texture = GetTargetTexture(context, target);
 
             if (!texture)
             {
@@ -5207,7 +5148,7 @@
                 return;
             }
 
-            gl::Texture *texture = getTargetTexture(context, target);
+            gl::Texture *texture = GetTargetTexture(context, target);
 
             if (!texture)
             {
@@ -5267,7 +5208,7 @@
                 return;
             }
 
-            gl::Texture *texture = getTargetTexture(context, target);
+            gl::Texture *texture = GetTargetTexture(context, target);
 
             if (!texture)
             {