Enable off-by-default monitor color management on Windows. This is enabled vi...

chrome/browser/renderer_host/backing_store_win.cc

 // Copyright (c) 2006-2008 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/renderer_host/backing_store.h"
 
+#include "base/command_line.h"
 #include "base/gfx/gdi_util.h"
 #include "chrome/browser/renderer_host/render_widget_host.h"
+#include "chrome/common/chrome_switches.h"
 #include "chrome/common/transport_dib.h"
 
 namespace {
 
 // Creates a dib conforming to the height/width/section parameters passed in.
 HANDLE CreateDIB(HDC dc, int width, int height, int color_depth) {
-  BITMAPINFOHEADER hdr;
-  gfx::CreateBitmapHeaderWithColorDepth(width, height, color_depth, &hdr);
+  BITMAPV5HEADER hdr = {0};
+  ZeroMemory(&hdr, sizeof(BITMAPV5HEADER));
+
+  // These values are shared with gfx::PlatformDevice
+  hdr.bV5Size = sizeof(BITMAPINFOHEADER);
+  hdr.bV5Width = width;
+  hdr.bV5Height = -height;  // minus means top-down bitmap
+  hdr.bV5Planes = 1;
+  hdr.bV5BitCount = color_depth;
+  hdr.bV5Compression = BI_RGB;  // no compression
+  hdr.bV5SizeImage = 0;
+  hdr.bV5XPelsPerMeter = 1;
+  hdr.bV5YPelsPerMeter = 1;
+  hdr.bV5ClrUsed = 0;

[M-A Ruel, 2009/06/18 00:19:09]

nit: Do you really want to keep the = 0?

+  hdr.bV5ClrImportant = 0;
+
+  if (BackingStore::ColorManagementEnabled()) {
+    hdr.bV5CSType = LCS_sRGB;
+    hdr.bV5Intent = LCS_GM_IMAGES;
+  }
+
   void* data = NULL;
   HANDLE dib = CreateDIBSection(dc, reinterpret_cast<BITMAPINFO*>(&hdr),
                                 0, &data, NULL, 0);
   DCHECK(data);
   return dib;
 }
 
 }  // namespace
 
 // BackingStore (Windows) ------------------------------------------------------
@@ skipping 19 matching lines @@
   if (original_bitmap_) {
     SelectObject(hdc_, original_bitmap_);
   }
   if (backing_store_dib_) {
     DeleteObject(backing_store_dib_);
     backing_store_dib_ = NULL;
   }
   DeleteDC(hdc_);
 }
 
+// static
+bool BackingStore::ColorManagementEnabled() {
+  static bool enabled = false;

[M-A Ruel, 2009/06/18 00:19:09]

humm...

There's one issue I easily see is RDP session. I guess this function is called
very often so it'd be costly to be looking at the command line at each call. I
still don't like static local variable.

+  static bool checked = false;
+  if (!checked) {
+    checked = true;
+    const CommandLine& command = *CommandLine::ForCurrentProcess();
+    enabled = command.HasSwitch(switches::kEnableMonitorProfile);
+  }
+  return enabled;
+}
+
 void BackingStore::PaintRect(base::ProcessHandle process,
                              TransportDIB* bitmap,
                              const gfx::Rect& bitmap_rect) {
   if (!backing_store_dib_) {
     backing_store_dib_ = CreateDIB(hdc_, size_.width(),
                                    size_.height(), color_depth_);
     if (!backing_store_dib_) {
       NOTREACHED();
       return;
     }
@@ skipping 27 matching lines @@
   ScrollDC(hdc_, dx, dy, NULL, &r, NULL, &damaged_rect);
 
   // TODO(darin): this doesn't work if dx and dy are both non-zero!
   DCHECK(dx == 0 || dy == 0);
 
   // We expect that damaged_rect should equal bitmap_rect.
   DCHECK(gfx::Rect(damaged_rect) == bitmap_rect);
 
   PaintRect(process, bitmap, bitmap_rect);
 }