Let cmd-` switch windows again.

chrome/browser/tab_contents/tab_contents_view_mac.mm

 // Copyright (c) 2006-2008 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.
 
+#import <Carbon/Carbon.h>
+
 #include "chrome/browser/tab_contents/tab_contents_view_mac.h"
 
 #include <string>
 
 #include "chrome/browser/browser.h" // TODO(beng): this dependency is awful.
 #import "chrome/browser/cocoa/focus_tracker.h"
 #import "chrome/browser/cocoa/chrome_browser_window.h"
 #import "chrome/browser/cocoa/browser_window_controller.h"
 #include "chrome/browser/global_keyboard_shortcuts_mac.h"
 #include "chrome/browser/cocoa/sad_tab_view.h"
@@ skipping 237 matching lines @@
         [sad_tab_.get() removeFromSuperview];
         sad_tab_.reset();
       }
       break;
     }
     default:
       NOTREACHED() << "Got a notification we didn't register for.";
   }
 }
 
+@interface NSApplication(SPI)
+- (void)_cycleWindowsReversed:(BOOL)reversed;

[James Su, 2009/10/15 08:00:49]

Is it ok for us to depend on a private method? It makes me a little nervous. And
what's the difference between _cycleWindows, _cycleWindowsBackwards and
_cycleWindowsReversed?

[Nico, 2009/10/15 15:40:00]

I think it's ok for often-used stuff as long as we check that the function
exists before calling it – that way we don't crash but only lose some
functionality if it goes away. Which we will notice, because when something
that's used often stops working, someone will notice. We do this in several
places already.

WebKit uses this function too (
https://svn.webkit.org/browser/trunk/WebKit/mac/DefaultDelegates/WebDefaultUI...
) and has for years, so this going away all of a sudden is unlikely.

_cycleWindowsReversed: receives a bool argument, while _cycleWindows: and
_cycleWindowsBackwards: seem to receive an (id)sender, so that the latter two
can be used as actions.

+@end
+
 @implementation TabContentsViewCocoa
 
 - (id)initWithTabContentsViewMac:(TabContentsViewMac*)w {
   self = [super initWithFrame:NSZeroRect];
   if (self != nil) {
     tabContentsView_ = w;
     dropTarget_.reset(
         [[WebDropTarget alloc] initWithTabContents:[self tabContents]]);
     [self registerDragTypes];
   }
@@ skipping 24 matching lines @@
 
 - (void)processKeyboardEvent:(NativeWebKeyboardEvent*)wkEvent {
   NSEvent* event = wkEvent->os_event;
 
   if ([event type] == NSKeyDown && ([event modifierFlags] & NSCommandKeyMask)) {
     // We need to dispatch this to the menu.
     if ([[NSApp mainMenu] performKeyEquivalent:event])
       return;
   }
 
+  // Cmd-` is not in the menu and it's apparently handled by |NSApp sendEvent|
+  // if the application doesn't swallow it. We do, so we need to handle this
+  // key ourself. On foreign keyboards, the "switch windows" key is not the
+  // ` key, so do this by keycode instead of |event characters|.

[James Su, 2009/10/15 08:00:49]

Is Cmd-` the only key handled by |NSApp sendEvent| but not in the menu? I feel
that the right approach is to feed the key event back to NSApp, so that we don't
need to care about how to handle it.

[Nico, 2009/10/15 15:40:00]

Experimental evidence suggests that it really is the only key, but I don't know.
(By "experimental evidence" I mean that it's the only key we received bugs for,
and it's also the only key that needed special handling in other projects I've
seen).

+  if ([event type] == NSKeyDown &&
+      [event keyCode] == kVK_ANSI_Grave &&
+      [NSApp respondsToSelector:@selector(_cycleWindowsReversed:)]) {
+    const NSUInteger kModifierMask = NSShiftKeyMask |
+                                     NSControlKeyMask |
+                                     NSAlternateKeyMask |
+                                     NSCommandKeyMask;
+    if (([event modifierFlags] & kModifierMask) == NSCommandKeyMask)
+      [NSApp _cycleWindowsReversed:NO];
+    else if (([event modifierFlags] & kModifierMask) ==
+        (NSCommandKeyMask | NSShiftKeyMask) &&
+        [NSApp respondsToSelector:@selector(_cycleWindowsReversed:)])
+      [NSApp _cycleWindowsReversed:YES];
+  }
+
   // If this tab is no longer active, it's window will be |nil|. In that case,
   // best ignore the event.
   if (![self window])
     return;
 
   // Do not fire shortcuts on key up.
   if ([event type] == NSKeyDown) {
     ChromeBrowserWindow* window = (ChromeBrowserWindow*)[self window];
     DCHECK([window isKindOfClass:[ChromeBrowserWindow class]]);
     if ([window handleExtraBrowserKeyboardShortcut:event])
@@ skipping 113 matching lines @@
 }
 
 - (BOOL)performDragOperation:(id<NSDraggingInfo>)sender {
   return [dropTarget_ performDragOperation:sender view:self];
 }
 
 // Tons of stuff goes here, where we grab events going on in Cocoaland and send
 // them into the C++ system. TODO(avi): all that jazz
 
 @end