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

Unified Diff: ui/views/controls/menu/menu_item_view.cc

Issue 2382443007: Clean up NativeTheme (particularly CommonTheme). (Closed)
Patch Set: tdandersonreview 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/views/controls/menu/menu_item_view.cc
diff --git a/ui/views/controls/menu/menu_item_view.cc b/ui/views/controls/menu/menu_item_view.cc
index d36350e12246dc611b8de88afb5a0ec8f7d6755a..a405cb24ced6de584d4147734675ef78c1d43e48 100644
--- a/ui/views/controls/menu/menu_item_view.cc
+++ b/ui/views/controls/menu/menu_item_view.cc
@@ -787,9 +787,9 @@ void MenuItemView::PaintButton(gfx::Canvas* canvas, PaintButtonMode mode) {
bool emphasized = delegate &&
delegate->GetShouldUseDisabledEmphasizedForegroundColor(
GetCommand());
- color_id = emphasized ?
- ui::NativeTheme::kColorId_DisabledEmphasizedMenuItemForegroundColor :
- ui::NativeTheme::kColorId_DisabledMenuItemForegroundColor;
+ color_id = emphasized
+ ? ui::NativeTheme::kColorId_EnabledMenuItemForegroundColor
+ : ui::NativeTheme::kColorId_DisabledMenuItemForegroundColor;
}
SkColor fg_color = native_theme->GetSystemColor(color_id);
SkColor override_foreground_color;

Powered by Google App Engine
This is Rietveld 408576698