commit ca27139e6bdcca9afe8c041ee3cb8b5f57c51b38
author Geoff Lang <geofflang@chromium.org> Wed Apr 05 12:30:00 2017 -0400
committer Commit Bot <commit-bot@chromium.org> Thu Apr 20 14:50:44 2017 +0000
tree df17073a69e1faad9fda4e2dca5f73f2fce3c15f
parent 8162926b2bbaba65cb5bb114403b4dac5b0677fe

Key the format tables on internal format and type.

Keying the format tables on internal format alone is not enough to fully
validate the unsized formats which require additional type information.

This CL has no functional changes, it just splits the tables and updates
the calls to GetInternalFormat info to provide type information when the
format is not sized.

BUG=angleproject:1523
BUG=angleproject:1958
BUG=angleproject:1228

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

src/libANGLE/renderer/renderer_utils.cpp

diff --git a/src/libANGLE/renderer/renderer_utils.cpp b/src/libANGLE/renderer/renderer_utils.cpp
index cc04dbb..9d0bf0e 100644
--- a/src/libANGLE/renderer/renderer_utils.cpp
+++ b/src/libANGLE/renderer/renderer_utils.cpp
@@ -187,7 +187,7 @@
         inputPitch = -inputPitch;
     }
 
-    const auto &sourceGLInfo = gl::GetInternalFormatInfo(sourceFormat.glInternalFormat);
+    const auto &sourceGLInfo = gl::GetSizedInternalFormatInfo(sourceFormat.glInternalFormat);
 
     if (sourceGLInfo.format == params.format && sourceGLInfo.type == params.type)
     {
@@ -200,13 +200,12 @@
         return;
     }
 
-    ASSERT(sourceGLInfo.pixelBytes > 0);
+    ASSERT(sourceGLInfo.sized);
 
     gl::FormatType formatType(params.format, params.type);
     ColorCopyFunction fastCopyFunc =
         GetFastCopyFunction(sourceFormat.fastCopyFunctions, formatType);
-    GLenum sizedDestInternalFormat = gl::GetSizedInternalFormat(formatType.format, formatType.type);
-    const auto &destFormatInfo     = gl::GetInternalFormatInfo(sizedDestInternalFormat);
+    const auto &destFormatInfo = gl::GetInternalFormatInfo(formatType.format, formatType.type);
 
     if (fastCopyFunc)
     {