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

Unified Diff: ui/compositor/compositor.cc

Issue 2735943002: cc: Clarify that color spaces are for raster (Closed)
Patch Set: Rebase (a few times) 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
« no previous file with comments | « content/renderer/render_widget.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ui/compositor/compositor.cc
diff --git a/ui/compositor/compositor.cc b/ui/compositor/compositor.cc
index f438b216f1d17ee15a8929d81267630ef372030d..a80a83bb7f8f6adda62e455cd8e86f4fa9f94240 100644
--- a/ui/compositor/compositor.cc
+++ b/ui/compositor/compositor.cc
@@ -161,11 +161,11 @@ Compositor::Compositor(const cc::FrameSinkId& frame_sink_id,
settings.use_layer_lists =
command_line->HasSwitch(cc::switches::kUIEnableLayerLists);
- settings.enable_color_correct_rendering =
+ settings.enable_color_correct_rasterization =
command_line->HasSwitch(cc::switches::kEnableColorCorrectRendering) ||
command_line->HasSwitch(cc::switches::kEnableTrueColorRendering);
settings.renderer_settings.enable_color_correct_rendering =
- settings.enable_color_correct_rendering ||
+ settings.enable_color_correct_rasterization ||
command_line->HasSwitch(switches::kEnableHDROutput);
// UI compositor always uses partial raster if not using zero-copy. Zero copy
@@ -362,7 +362,7 @@ void Compositor::SetDisplayColorSpace(const gfx::ColorSpace& color_space) {
blending_color_space_ = gfx::ColorSpace::CreateExtendedSRGB();
output_color_space_ = gfx::ColorSpace::CreateSCRGBLinear();
}
- host_->SetDeviceColorSpace(blending_color_space_);
+ host_->SetRasterColorSpace(color_space);
// Color space is reset when the output surface is lost, so this must also be
// updated then.
// TODO(fsamuel): Get rid of this.
« no previous file with comments | « content/renderer/render_widget.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698