Add a new GetSystemColor method to native theme.

ui/gfx/native_theme_win.cc

 // Copyright (c) 2011 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 "ui/gfx/native_theme_win.h"
 
 #include <windows.h>
 #include <uxtheme.h>
 #include <vsstyle.h>
 #include <vssym32.h>
 
 #include "base/basictypes.h"
 #include "base/logging.h"
 #include "base/memory/scoped_handle.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/win/scoped_gdi_object.h"
 #include "base/win/scoped_hdc.h"
 #include "base/win/scoped_select_object.h"
 #include "base/win/windows_version.h"
 #include "skia/ext/platform_canvas.h"
 #include "skia/ext/skia_utils_win.h"
 #include "third_party/skia/include/core/SkCanvas.h"
 #include "third_party/skia/include/core/SkColorPriv.h"
 #include "third_party/skia/include/core/SkShader.h"
 #include "ui/gfx/gdi_util.h"
 #include "ui/gfx/rect.h"
 
 namespace {
 
+const SkColor kInvalidColorIdColor = SkColorSetRGB(255, 0, 128);
+
+SkColor WinColorToSkColor(COLORREF color) {
+  return SkColorSetRGB(GetRValue(color), GetGValue(color), GetBValue(color));
+}
+
 void SetCheckerboardShader(SkPaint* paint, const RECT& align_rect) {
   // Create a 2x2 checkerboard pattern using the 3D face and highlight colors.
   SkColor face = skia::COLORREFToSkColor(GetSysColor(COLOR_3DFACE));
   SkColor highlight = skia::COLORREFToSkColor(GetSysColor(COLOR_3DHILIGHT));
   SkColor buffer[] = { face, highlight, highlight, face };
   // Confusing bit: we first create a temporary bitmap with our desired pattern,
   // then copy it to another bitmap.  The temporary bitmap doesn't take
   // ownership of the pixel data, and so will point to garbage when this
   // function returns.  The copy will copy the pixel data into a place owned by
   // the bitmap, which is in turn owned by the shader, etc., so it will live
@@ skipping 1635 matching lines @@
           break;
       }
       break;
     default:
       NOTREACHED() << "Invalid part: " << part;
       break;
   }
   return state_id;
 }
 
+SkColor NativeThemeWin::GetSystemColor(ColorId color_id) const {

[Ben Goodger (Google), 2011/11/21 18:59:59]

and update .cc to match .h file location

[benrg, 2011/11/21 19:54:20]

Done, but this puts GetSystemColor in the middle of the part-related methods. I
wonder if I should move GetSystemColor to the top everywhere to avoid that.

+  switch (color_id) {
+    case kDialogBackgroundColorId:
+      // TODO(benrg): Should this use the new Windows theme functions? The old
+      // code in DialogClientView::OnPaint used GetSysColor(COLOR_3DFACE).
+      return WinColorToSkColor(GetSysColor(COLOR_3DFACE));
+    default:
+      NOTREACHED() << "Invalid color_id: " << color_id;
+      break;
+  }
+  return kInvalidColorIdColor;
+}
+
 }  // namespace gfx