Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(322)

Unified Diff: src/gpu/gl/GrGLCaps.cpp

Issue 1266933002: Disable SRGB support on PowerVR Rogue due to SRGBReadWritePixels failure (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: fix mising paren Created 5 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/gpu/gl/GrGLCaps.cpp
diff --git a/src/gpu/gl/GrGLCaps.cpp b/src/gpu/gl/GrGLCaps.cpp
index 161dac6e9f3bb2ed988cd2ca458eee2003782da5..b0f6f2fcf7dcb1323a281122e673c557c4b8410b 100644
--- a/src/gpu/gl/GrGLCaps.cpp
+++ b/src/gpu/gl/GrGLCaps.cpp
@@ -206,8 +206,9 @@ void GrGLCaps::init(const GrContextOptions& contextOptions,
// All the above srgb extensions support toggling srgb writes
fSRGBWriteControl = srgbSupport;
} else {
- srgbSupport = ctxInfo.version() >= GR_GL_VER(3,0) ||
- ctxInfo.hasExtension("GL_EXT_sRGB");
+ // See http://skbug.com/4148 for PowerVR issue.
+ srgbSupport = kPowerVRRogue_GrGLRenderer != ctxInfo.renderer() &&
+ (ctxInfo.version() >= GR_GL_VER(3,0) || ctxInfo.hasExtension("GL_EXT_sRGB"));
// ES through 3.1 requires EXT_srgb_write_control to support toggling
// sRGB writing for destinations.
fSRGBWriteControl = ctxInfo.hasExtension("GL_EXT_sRGB_write_control");
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698