Fix accelerator handling for in-menu buttons in the app menu.

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 <algorithm>
 #include <cmath>
 #include <set>
 
@@ skipping 283 matching lines @@
           views::Button::STATE_PRESSED,
           theme->GetSystemColor(
               ui::NativeTheme::kColorId_SelectedMenuItemForegroundColor));
       SetTextColor(
           views::Button::STATE_NORMAL,
           theme->GetSystemColor(
               ui::NativeTheme::kColorId_EnabledMenuItemForegroundColor));
     }
   }
 
+  // views::CustomButton
+  bool ShouldHandleAcceleratorWhenInactive() override {
+    // Menu buttons are inactive, so handle accelerator presses when inactive.
+    return true;
+  }
+
  private:
   InMenuButtonBackground* in_menu_background_;
 
   DISALLOW_COPY_AND_ASSIGN(InMenuButton);
 };
 
 // AppMenuView is a view that can contain label buttons.
 class AppMenuView : public views::View,
                     public views::ButtonListener,
                     public AppMenuObserver {
@@ skipping 948 matching lines @@
                                 0,
                                 BookmarkMenuDelegate::SHOW_PERMANENT_FOLDERS,
                                 BOOKMARK_LAUNCH_LOCATION_WRENCH_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;
 }