commit 586bc55cafa31ca6cfb19e7e50526d32d4d58396
author Olli Etuaho <oetuaho@nvidia.com> Fri Mar 04 11:46:03 2016 +0200
committer Olli Etuaho <oetuaho@nvidia.com> Sat Mar 05 08:34:51 2016 +0000
tree 6691ce47b7b8ad1671c8862739a2448bfe8f5db1
parent e18eb970f23ba5e8d06def088bbd5f312ceb0351

Return the correct count from getAttachedShaders

The count should be the total amount of shader ids written. Before
this patch the code used to write the amount of attached shaders
to count regardless of the value of maxCount.

BUG=angleproject:1101
TEST=dEQP-GLES3.functional.state_query.shader.*

Change-Id: I5ae729748c7809f03fd496c927a76f60398d2b42
Reviewed-on: https://chromium-review.googlesource.com/329865
Reviewed-by: Corentin Wallez <cwallez@chromium.org>
Commit-Queue: Olli Etuaho <oetuaho@nvidia.com>
Reviewed-by: Jamie Madill <jmadill@chromium.org>

src/libANGLE/Program.cpp

diff --git a/src/libANGLE/Program.cpp b/src/libANGLE/Program.cpp
index 748ceae..bd16fd9 100644
--- a/src/libANGLE/Program.cpp
+++ b/src/libANGLE/Program.cpp
@@ -859,9 +859,8 @@
         if (total < maxCount)
         {
             shaders[total] = mData.mAttachedVertexShader->getHandle();
+            total++;
         }
-
-        total++;
     }
 
     if (mData.mAttachedFragmentShader)
@@ -869,9 +868,8 @@
         if (total < maxCount)
         {
             shaders[total] = mData.mAttachedFragmentShader->getHandle();
+            total++;
         }
-
-        total++;
     }
 
     if (count)