commit 9fdaa497b489b7e7739e29812c85da2125b6a681
author Jamie Madill <jmadill@chromium.org> Fri Feb 16 10:52:11 2018 -0500
committer Commit Bot <commit-bot@chromium.org> Sat Feb 17 19:38:59 2018 +0000
tree 01bd0a926d7b14ad681c3fb42efffb72271b7181
parent 0cbfa5866f43b6098a8d963ca9ac6f4a096bf42c

Don't no-op draw calls for zero count in validation.

Make the no-op happen in the Context, so we can properly generator
more errors for negative WebGL tests. Some validation tests still
need to check for no-ops, such as range checks.

BUG=angleproject:2050

Change-Id: I48d0b3cf640f7f128679600e5df108146cfd6348
Reviewed-on: https://chromium-review.googlesource.com/522869
Commit-Queue: Jamie Madill <jmadill@chromium.org>
Reviewed-by: Corentin Wallez <cwallez@chromium.org>

src/libANGLE/validationES3.cpp

diff --git a/src/libANGLE/validationES3.cpp b/src/libANGLE/validationES3.cpp
index 2e0c02f..0aa209d 100644
--- a/src/libANGLE/validationES3.cpp
+++ b/src/libANGLE/validationES3.cpp
@@ -1276,6 +1276,12 @@
         return false;
     }
 
+    // Skip range checks for no-op calls.
+    if (count <= 0)
+    {
+        return true;
+    }
+
     // Use the parameter buffer to retrieve and cache the index range.
     const auto &params        = context->getParams<HasIndexRange>();
     const auto &indexRangeOpt = params.getIndexRange();