Changes menu button to not show the menu on alt-space. alt-space

views/controls/button/menu_button.cc

 // Copyright (c) 2011 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 "views/controls/button/menu_button.h"
 
 #include "base/utf_string_conversions.h"
 #include "grit/app_strings.h"
 #include "grit/ui_resources.h"
 #include "ui/base/accessibility/accessible_view_state.h"
@@ skipping 203 matching lines @@
 // cause the button to appear depressed while the menu is displayed.
 void MenuButton::OnMouseExited(const MouseEvent& event) {
   if ((state_ != BS_DISABLED) && (!menu_visible_) && (!InDrag())) {
     SetState(BS_NORMAL);
   }
 }
 
 bool MenuButton::OnKeyPressed(const KeyEvent& event) {
   switch (event.key_code()) {
     case ui::VKEY_SPACE:
+      if (event.IsAltDown()) {
+        // alt-space on windows should show the window menu.

[Peter Kasting, 2011/07/06 17:47:46]

Tiny nit: Capitalize!

(Also, this would be shorter and maybe fractionally better if the comment went
above the conditional, and the {} disappeared.)

+        break;
+      }
     case ui::VKEY_RETURN:
     case ui::VKEY_UP:
     case ui::VKEY_DOWN: {
       bool result = Activate();
       if (GetFocusManager()->GetFocusedView() == NULL)
         RequestFocus();
       return result;
     }
     default:
       break;
@@ skipping 20 matching lines @@
     NOTREACHED();
     return 0;
   }
 
   gfx::Rect monitor_bounds =
       Screen::GetMonitorWorkAreaNearestWindow(GetWidget()->GetNativeView());
   return monitor_bounds.right() - 1;
 }
 
 }  // namespace views