Index: chrome/browser/ui/views/toolbar/app_menu.cc |
diff --git a/chrome/browser/ui/views/toolbar/app_menu.cc b/chrome/browser/ui/views/toolbar/app_menu.cc |
index 6d940d5cfc20af821d3f82e05c2a4de57d575f87..c55221dff45d1be80965489fa9e874625ef6bb93 100644 |
--- a/chrome/browser/ui/views/toolbar/app_menu.cc |
+++ b/chrome/browser/ui/views/toolbar/app_menu.cc |
@@ -857,7 +857,9 @@ const gfx::FontList* AppMenu::GetLabelFontList(int command_id) const { |
} |
bool AppMenu::GetShouldUseNormalForegroundColor(int command_id) const { |
- return IsRecentTabsCommand(command_id); |
+ // Use the normal foreground color instead of the disabled color for the |
+ // recent tab headers. Only the headers from that submenu have font lists. |
+ return IsRecentTabsCommand(command_id) && GetLabelFontList(command_id); |
Evan Stade
2017/02/16 01:30:04
this may seem a little odd but it's what the code
msw
2017/02/16 01:54:00
Can we omit the recently closed item when there ar
Evan Stade
2017/02/16 02:36:25
I think it's better to show it but have it disable
msw
2017/02/16 04:18:08
Acknowledged.
|
} |
base::string16 AppMenu::GetTooltipText(int command_id, |