Don't enable/disable sRGB on GLES.
glEnable/glDisable GL_FRAMEBUFFER_SRGB isn't meaningful on GLES.
BUG=angleproject:1300
Change-Id: I0afd697ed9770e2eb1cdcc851884b90de1a4583c
Reviewed-on: https://chromium-review.googlesource.com/334490
Reviewed-by: Jamie Madill <jmadill@chromium.org>
Reviewed-by: Geoff Lang <geofflang@chromium.org>
Commit-Queue: Frank Henigman <fjhenigman@chromium.org>
diff --git a/src/libANGLE/renderer/gl/FramebufferGL.cpp b/src/libANGLE/renderer/gl/FramebufferGL.cpp
index 566de45..6c9f046 100644
--- a/src/libANGLE/renderer/gl/FramebufferGL.cpp
+++ b/src/libANGLE/renderer/gl/FramebufferGL.cpp
@@ -336,24 +336,27 @@
void FramebufferGL::syncClearState(GLbitfield mask)
{
- if (mWorkarounds.doesSRGBClearsOnLinearFramebufferAttachments &&
- (mask & GL_COLOR_BUFFER_BIT) != 0 && !mIsDefault)
+ if (mFunctions->standard == STANDARD_GL_DESKTOP)
{
- bool hasSRBAttachment = false;
- for (const auto &attachment : mData.getColorAttachments())
+ if (mWorkarounds.doesSRGBClearsOnLinearFramebufferAttachments &&
+ (mask & GL_COLOR_BUFFER_BIT) != 0 && !mIsDefault)
{
- if (attachment.isAttached() && attachment.getColorEncoding() == GL_SRGB)
+ bool hasSRBAttachment = false;
+ for (const auto &attachment : mData.getColorAttachments())
{
- hasSRBAttachment = true;
- break;
+ if (attachment.isAttached() && attachment.getColorEncoding() == GL_SRGB)
+ {
+ hasSRBAttachment = true;
+ break;
+ }
}
- }
- mStateManager->setFramebufferSRGBEnabled(hasSRBAttachment);
- }
- else
- {
- mStateManager->setFramebufferSRGBEnabled(!mIsDefault);
+ mStateManager->setFramebufferSRGBEnabled(hasSRBAttachment);
+ }
+ else
+ {
+ mStateManager->setFramebufferSRGBEnabled(!mIsDefault);
+ }
}
}