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

Unified Diff: cc/output/gl_renderer.cc

Issue 2769863003: cc: Add pixel tests for transfer function shaders (Closed)
Patch Set: More comments Created 3 years, 9 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
Index: cc/output/gl_renderer.cc
diff --git a/cc/output/gl_renderer.cc b/cc/output/gl_renderer.cc
index 809bbdd130ed1b838241fe0de5511f3029772587..a9b954d2ab6a7537746da0ba7a8c73c0ce0b7a7b 100644
--- a/cc/output/gl_renderer.cc
+++ b/cc/output/gl_renderer.cc
@@ -3004,11 +3004,11 @@ void GLRenderer::SetUseProgram(const ProgramKey& program_key,
// https://crbug.com/699243
// The source color space for non-YUV draw quads should always be full-range
// RGB.
- DCHECK_EQ(src_color_space, src_color_space.GetAsFullRangeRGB());
if (settings_->enable_color_correct_rendering) {
SetUseProgram(program_key, src_color_space,
current_frame()->current_render_pass->color_space);
} else {
ccameron 2017/03/23 06:41:40 Moving this allows me to test YUV shaders in the s
hubbe 2017/03/23 18:09:33 I think this check is iffy anyways. :)
+ DCHECK_EQ(src_color_space, src_color_space.GetAsFullRangeRGB());
SetUseProgram(program_key, gfx::ColorSpace(), gfx::ColorSpace());
}
}

Powered by Google App Engine
This is Rietveld 408576698