Index: tests/SRGBMipMapTest.cpp |
diff --git a/tests/SRGBMipMapTest.cpp b/tests/SRGBMipMapTest.cpp |
index 88f08d43d73e77a214f55bfd775608ef76f63f1a..b963aaff4b34c22cc4bc80591264f359591cfe8a 100644 |
--- a/tests/SRGBMipMapTest.cpp |
+++ b/tests/SRGBMipMapTest.cpp |
@@ -148,10 +148,14 @@ DEF_GPUTEST_FOR_GL_RENDERING_CONTEXTS(SRGBMipMaps, reporter, ctxInfo) { |
// skbug.com/5048). On GL, we may not have sRGB decode support. In that case, rendering sRGB |
// textures to a legacy surface produces nonsense, so this part of the test is meaningless. |
// |
+ // We also skip this part of the test on command buffer (via srgbDecodeDisableAffectsMipmaps), |
+ // because that implementation of the extension doesn't ensure that mips respect the setting. |
+ // |
// TODO: Once Vulkan supports legacy mip-mapping, we can promote this to GrCaps. Right now, |
// Vulkan has most of the functionality, but not the mip-mapping part that's being tested here. |
GrGLGpu* glGpu = static_cast<GrGLGpu*>(context->getGpu()); |
- if (glGpu->glCaps().srgbDecodeDisableSupport()) { |
+ if (glGpu->glCaps().srgbDecodeDisableSupport() && |
+ glGpu->glCaps().srgbDecodeDisableAffectsMipmaps()) { |
read_and_check_pixels(reporter, l32RenderTargetContext->asTexture().get(), expectedLinear, |
error, "re-render as linear"); |
} |