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

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

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/ui/views/toolbar/app_menu.h"
 
 #include <stdint.h>
 
 #include <algorithm>
 #include <cmath>
@@ skipping 839 matching lines @@
 
 const gfx::FontList* AppMenu::GetLabelFontList(int command_id) const {
   if (IsRecentTabsCommand(command_id)) {
     return recent_tabs_menu_model_delegate_->GetLabelFontListAt(
         ModelIndexFromCommandId(command_id));
   }
   return NULL;
 }
 
 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,
                                        const gfx::Point& p) const {
   return IsBookmarkCommand(command_id) ?
       bookmark_menu_delegate_->GetTooltipText(command_id, p) : base::string16();
 }
 
 bool AppMenu::IsTriggerableEvent(views::MenuItemView* menu,
                                  const ui::Event& e) {
@@ skipping 363 matching lines @@
                                 0,
                                 BookmarkMenuDelegate::SHOW_PERMANENT_FOLDERS,
                                 BOOKMARK_LAUNCH_LOCATION_APP_MENU);
 }
 
 int AppMenu::ModelIndexFromCommandId(int command_id) const {
   CommandIDToEntry::const_iterator ix = command_id_to_entry_.find(command_id);
   DCHECK(ix != command_id_to_entry_.end());
   return ix->second.second;
 }