Don't use bold text for the "no recently closed tabs" menu item.

chrome/browser/ui/views/toolbar/app_menu.cc

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 was doing before I touched it
and I don't see a more elegant solution.

[msw, 2017/02/16 01:54:00]

Can we omit the recently closed item when there are no sub-items?

[Evan Stade, 2017/02/16 02:36:25]

I think it's better to show it but have it disabled. If you're actually looking
for it and it's just not there, you might think you're looking in the wrong
place.

[msw, 2017/02/16 04:18:08]

Acknowledged.

 }
 
 base::string16 AppMenu::GetTooltipText(int command_id,