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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: ui/native_theme/native_theme.h

Issue 12483006: wrench_menu: use NativeTheme colors under aura (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: no need to pass View from toolbarview to wrenchmenu Created 7 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
« chrome/browser/ui/views/wrench_menu.cc ('K') | « ui/native_theme/common_theme.cc ('k') | ui/native_theme/native_theme_aura.cc » ('j') | ui/native_theme/native_theme_gtk.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ui/native_theme/native_theme.h
diff --git a/ui/native_theme/native_theme.h b/ui/native_theme/native_theme.h
index 2958609dda7f76cb815080765a378c833f925b27..b60bb1c87b0628089a69c1f515abfd4aead17d10 100644
--- a/ui/native_theme/native_theme.h
+++ b/ui/native_theme/native_theme.h
@@ -240,9 +240,13 @@ class NATIVE_THEME_EXPORT NativeTheme {
kColorId_EnabledMenuItemForegroundColor,
kColorId_DisabledMenuItemForegroundColor,
kColorId_FocusedMenuItemBackgroundColor,
+ kColorId_HoverMenuItemBackgroundColor,
kColorId_MenuSeparatorColor,
kColorId_MenuBackgroundColor,
kColorId_MenuBorderColor,
+ kColorId_EnabledMenuButtonBorderColor,
sky 2013/03/20 16:30:02 It isn't obvious what menu button border is. Add a
Bobby Powers 2013/03/21 17:08:43 Done.
+ kColorId_FocusedMenuButtonBorderColor,
+ kColorId_HoverMenuButtonBorderColor,
// Label
kColorId_LabelEnabledColor,
kColorId_LabelDisabledColor,
« chrome/browser/ui/views/wrench_menu.cc ('K') | « ui/native_theme/common_theme.cc ('k') | ui/native_theme/native_theme_aura.cc » ('j') | ui/native_theme/native_theme_gtk.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698