commit 59c41597b45fbad61c0b624b125cb6c3dfd9cd63
author Corentin Wallez <cwallez@chromium.org> Tue Jul 11 13:19:54 2017 -0400
committer Corentin Wallez <cwallez@chromium.org> Tue Jul 11 13:21:49 2017 -0400
tree 1bcdd7b2f0377a2977374f634dbb83537c79683f
parent 56229f1bbe2411571a4505e42fd8e58eb60a5552

Fix Clear validation assert for default FBOs

The validation was iterating over maxDrawBuffers attachments when
default framebuffers only have one attachment. Use the framebuffer's
drawBufferCount instead.

Also adds a regression test in the form of a WebGLComptibility test for
glClearBuffer with the default framebuffer.

BUG=angleproject:2091

Change-Id: I07ee524db1fcb8a99dab4043248c0885100fd216

src/libANGLE/validationES2.cpp

diff --git a/src/libANGLE/validationES2.cpp b/src/libANGLE/validationES2.cpp
index 9907e2a..a526aab 100644
--- a/src/libANGLE/validationES2.cpp
+++ b/src/libANGLE/validationES2.cpp
@@ -2302,7 +2302,7 @@
         constexpr GLenum validComponentTypes[] = {GL_FLOAT, GL_UNSIGNED_NORMALIZED,
                                                   GL_SIGNED_NORMALIZED};
 
-        for (GLuint drawBufferIdx = 0; drawBufferIdx < context->getCaps().maxDrawBuffers;
+        for (GLuint drawBufferIdx = 0; drawBufferIdx < fbo->getDrawbufferStateCount();
              drawBufferIdx++)
         {
             if (!ValidateWebGLFramebufferAttachmentClearType(