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

Unified Diff: ui/native_theme/native_theme_win.cc

Issue 2382443007: Clean up NativeTheme (particularly CommonTheme). (Closed)
Patch Set: two slight fixes Created 4 years, 3 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: ui/native_theme/native_theme_win.cc
diff --git a/ui/native_theme/native_theme_win.cc b/ui/native_theme/native_theme_win.cc
index f9cfe6074d8855757edca17ae74ef6e2b8d6bfdc..eb03ae51ad6f3595fbddc584441a787b4965da73 100644
--- a/ui/native_theme/native_theme_win.cc
+++ b/ui/native_theme/native_theme_win.cc
@@ -505,8 +505,6 @@ SkColor NativeThemeWin::GetSystemColor(ColorId color_id) const {
const SkColor kFocusedBorderColor = SkColorSetRGB(0x4d, 0x90, 0xfe);
const SkColor kUnfocusedBorderColor = SkColorSetRGB(0xd9, 0xd9, 0xd9);
// Button:
- const SkColor kButtonBackgroundColor = SkColorSetRGB(0xde, 0xde, 0xde);
- const SkColor kButtonHighlightColor = SkColorSetARGB(200, 255, 255, 255);
const SkColor kButtonHoverColor = SkColorSetRGB(6, 45, 117);
const SkColor kProminentButtonColorInvert = gfx::kGoogleBlue300;
// MenuItem:
@@ -541,12 +539,8 @@ SkColor NativeThemeWin::GetSystemColor(ColorId color_id) const {
return kUnfocusedBorderColor;
// Button
- case kColorId_ButtonBackgroundColor:
- return kButtonBackgroundColor;
case kColorId_ButtonEnabledColor:
return system_colors_[COLOR_BTNTEXT];
- case kColorId_ButtonHighlightColor:
- return kButtonHighlightColor;
case kColorId_ButtonHoverColor:
return kButtonHoverColor;
@@ -555,8 +549,6 @@ SkColor NativeThemeWin::GetSystemColor(ColorId color_id) const {
return system_colors_[COLOR_BTNTEXT];
case kColorId_LabelDisabledColor:
return system_colors_[COLOR_GRAYTEXT];
- case kColorId_LabelBackgroundColor:
- return system_colors_[COLOR_WINDOW];
// Textfield
case kColorId_TextfieldDefaultColor:

Powered by Google App Engine
This is Rietveld 408576698