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

Unified Diff: ui/native_theme/common_theme.cc

Issue 2616273002: Native Themes: Add table header colors (Closed)
Patch Set: Address sky@ and estade@'s comments Created 3 years, 11 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/common_theme.cc
diff --git a/ui/native_theme/common_theme.cc b/ui/native_theme/common_theme.cc
index f41e775465dbfc9d7f239159fa9444e5d0ba03b8..d0f749dd0c082c4a2aa3690a12186ca6383f466a 100644
--- a/ui/native_theme/common_theme.cc
+++ b/ui/native_theme/common_theme.cc
@@ -261,6 +261,14 @@ SkColor GetAuraColor(NativeTheme::ColorId color_id,
case NativeTheme::kColorId_TableGroupingIndicatorColor:
return kTableGroupingIndicatorColor;
+ // Table Header
+ case NativeTheme::kColorId_TableHeaderText:
+ return kEnabledMenuItemForegroundColor;
Evan Stade 2017/01/20 01:46:02 can you write this as base_theme->GetColor(MenuCol
Tom (Use chromium acct) 2017/01/20 19:10:38 Done.
+ case NativeTheme::kColorId_TableHeaderBackground:
+ return kMenuBackgroundColor;
+ case NativeTheme::kColorId_TableHeaderSeparator:
+ return kEnabledMenuButtonBorderColor;
+
// FocusableBorder
case NativeTheme::kColorId_FocusedBorderColor:
return gfx::kGoogleBlue500;

Powered by Google App Engine
This is Rietveld 408576698