Use active textures mask with robust init.

src/libANGLE/Context.cpp

@@ -2838,8 +2838,13 @@ void Context::initWorkarounds()
 Error Context::prepareForDraw()
 {
     syncRendererState();
-    ANGLE_TRY(mGLState.clearUnclearedActiveTextures(this));
-    ANGLE_TRY(mGLState.getDrawFramebuffer()->ensureDrawAttachmentsInitialized(this));
+
+    if (isRobustResourceInitEnabled())
+    {
+        ANGLE_TRY(mGLState.clearUnclearedActiveTextures(this));
+        ANGLE_TRY(mGLState.getDrawFramebuffer()->ensureDrawAttachmentsInitialized(this));
+    }
+
     return NoError();
 }
 
@@ -4283,7 +4288,10 @@ void Context::dispatchCompute(GLuint numGroupsX, GLuint numGroupsY, GLuint numGr
     }
 
     // TODO(jmadill): Dirty bits for compute.
-    ANGLE_CONTEXT_TRY(mGLState.clearUnclearedActiveTextures(this));
+    if (isRobustResourceInitEnabled())
+    {
+        ANGLE_CONTEXT_TRY(mGLState.clearUnclearedActiveTextures(this));
+    }
 
     handleError(mImplementation->dispatchCompute(this, numGroupsX, numGroupsY, numGroupsZ));
 }

src/libANGLE/State.cpp

@@ -2299,8 +2299,8 @@ void State::syncProgramTextures(const Context *context)
 
             // Bind the texture unconditionally, to recieve completeness change notifications.
             mCompleteTextureBindings[textureUnitIndex].bind(texture->getDirtyChannel());
+            mActiveTexturesMask.set(textureUnitIndex);
             newActiveTextures.set(textureUnitIndex);
-            mCompleteTexturesMask.set(textureUnitIndex);
 
             if (sampler != nullptr)
             {
@@ -2310,14 +2310,14 @@ void State::syncProgramTextures(const Context *context)
     }
 
     // Unset now missing textures.
-    ActiveTextureMask negativeMask = mCompleteTexturesMask & ~newActiveTextures;
+    ActiveTextureMask negativeMask = mActiveTexturesMask & ~newActiveTextures;
     if (negativeMask.any())
     {
         for (auto textureIndex : negativeMask)
         {
             mCompleteTextureBindings[textureIndex].reset();
             mCompleteTextureCache[textureIndex] = nullptr;
-            mCompleteTexturesMask.reset(textureIndex);
+            mActiveTexturesMask.reset(textureIndex);
         }
     }
 }
@@ -2422,19 +2422,17 @@ void State::signal(size_t textureIndex, InitState initState)
 
 Error State::clearUnclearedActiveTextures(const Context *context)
 {
-    if (!mRobustResourceInit)
-    {
-        return NoError();
-    }
+    ASSERT(mRobustResourceInit);
 
-    // TODO(jmadill): Investigate improving the speed here.
-    for (Texture *texture : mCompleteTextureCache)
+    for (auto textureIndex : mActiveTexturesMask)
     {
+        Texture *texture = mCompleteTextureCache[textureIndex];
         if (texture)
         {
             ANGLE_TRY(texture->ensureInitialized(context));
         }
     }
+
     return NoError();
 }
 

src/libANGLE/State.h

@@ -573,7 +573,7 @@ class State : public OnAttachmentDirtyReceiver, angle::NonCopyable
     std::vector<Texture *> mCompleteTextureCache;
     std::vector<OnAttachmentDirtyBinding> mCompleteTextureBindings;
     using ActiveTextureMask = angle::BitSet<IMPLEMENTATION_MAX_ACTIVE_TEXTURES>;
-    ActiveTextureMask mCompleteTexturesMask;
+    ActiveTextureMask mActiveTexturesMask;
 
     typedef std::vector<BindingPointer<Sampler>> SamplerBindingVector;
     SamplerBindingVector mSamplers;

src/tests/gl_tests/IncompleteTextureTest.cpp

@@ -103,14 +103,14 @@ TEST_P(IncompleteTextureTest, IncompleteTexture2D)
 
     // Should be complete - expect red.
     drawQuad(mProgram, "position", 0.5f);
-    EXPECT_PIXEL_COLOR_EQ(0, 0, GLColor::red);
+    EXPECT_PIXEL_COLOR_EQ(0, 0, GLColor::red) << "complete texture should be red";
 
     // Make texture incomplete.
     glTexParameteri(GL_TEXTURE_2D, GL_TEXTURE_MIN_FILTER, GL_LINEAR_MIPMAP_LINEAR);
 
     // Should be incomplete - expect black.
     drawQuad(mProgram, "position", 0.5f);
-    EXPECT_PIXEL_COLOR_EQ(0, 0, GLColor::black);
+    EXPECT_PIXEL_COLOR_EQ(0, 0, GLColor::black) << "incomplete texture should be black";
 
     // Make texture complete by defining the second mip.
     glTexImage2D(GL_TEXTURE_2D, 1, GL_RGBA, kTextureSize >> 1, kTextureSize >> 1, 0, GL_RGBA,
@@ -118,7 +118,7 @@ TEST_P(IncompleteTextureTest, IncompleteTexture2D)
 
     // Should be complete - expect red.
     drawQuad(mProgram, "position", 0.5f);
-    EXPECT_PIXEL_COLOR_EQ(0, 0, GLColor::red);
+    EXPECT_PIXEL_COLOR_EQ(0, 0, GLColor::red) << "mip-complete texture should be red";
 }
 
 // Tests redefining a texture with half the size works as expected.