Prevent extension and bookmark popups from interfering with close window/tab shortcuts.

chrome/browser/ui/cocoa/browser_window_controller.mm

Index: chrome/browser/ui/cocoa/browser_window_controller.mm
diff --git a/chrome/browser/ui/cocoa/browser_window_controller.mm b/chrome/browser/ui/cocoa/browser_window_controller.mm
index 84e053d230be987817e39c65ab082e31cf621a6e..9f2f4be4218ebbe2ad97211578425ce2e7dd9649 100644
--- a/chrome/browser/ui/cocoa/browser_window_controller.mm
+++ b/chrome/browser/ui/cocoa/browser_window_controller.mm
@@ -995,11 +995,6 @@ enum {
       // Generate return value (enabled state)
       enable = browser_->command_updater()->IsCommandEnabled(tag);
       switch (tag) {
-        case IDC_CLOSE_TAB:
-          // Disable "close tab" if we're not the key window or if there's only
-          // one tab.
-          enable &= [[self window] isKeyWindow];
-          break;

[Ilya Sherman, 2011/10/15 04:31:38]

I think this code is obsolete.  It definitely only does part of what the comment
says it does, whereas |fixCloseMenuItemKeyEquivalents| in app_controller_mac
does both.

         case IDC_FULLSCREEN: {
           enable &= [self supportsFullscreen];
           if ([static_cast<NSObject*>(item) isKindOfClass:[NSMenuItem class]]) {