[Mac] Factor out keyboard shortcut handling from ChromeEventProcessingWindow.

chrome/browser/ui/cocoa/chrome_command_dispatch_delegate.mm

-// Copyright (c) 2012 The Chromium Authors. All rights reserved.
+// Copyright 2015 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 "chrome/browser/ui/cocoa/chrome_event_processing_window.h"
+#import "chrome/browser/ui/cocoa/chrome_command_dispatch_delegate.h"
 
 #include "base/logging.h"
 #include "chrome/browser/global_keyboard_shortcuts_mac.h"
 #include "chrome/browser/ui/browser_commands.h"
 #include "chrome/browser/ui/browser_finder.h"
 #import "chrome/browser/ui/cocoa/browser_window_controller_private.h"
 #import "chrome/browser/ui/cocoa/tabs/tab_strip_controller.h"
 #import "content/public/browser/render_widget_host_view_mac_base.h"
 
 namespace {
@@ skipping 76 matching lines @@
                       windowNumber:[window windowNumber]
                            context:nil
                         characters:characters
        charactersIgnoringModifiers:charactors_ignoring_modifiers
                          isARepeat:is_a_repeat
                            keyCode:[event keyCode]];
 }
 
 }  // namespace
 
-@implementation ChromeEventProcessingWindow
+@implementation ChromeCommandDispatchDelegate
 
-- (BOOL)handleExtraKeyboardShortcut:(NSEvent*)event {
-  return HandleExtraBrowserKeyboardShortcut(event, self) ||
-         HandleExtraWindowKeyboardShortcut(event, self) ||
-         HandleDelayedWindowKeyboardShortcut(event, self);
+- (BOOL)handleExtraKeyboardShortcut:(NSEvent*)event window:(NSWindow*)window {
+  return HandleExtraBrowserKeyboardShortcut(event, window) ||
+         HandleExtraWindowKeyboardShortcut(event, window) ||
+         HandleDelayedWindowKeyboardShortcut(event, window);
 }
 
-- (BOOL)performKeyEquivalent:(NSEvent*)event {
+- (BOOL)performKeyEquivalent:(NSEvent*)event
+                      window:(NSWindow*)window

[tapted, 2015/07/31 03:06:49]

Could the delegate simply have a (weak) NSWindow* owner_; member?

[jackhou1, 2015/08/04 01:03:58]

Done.

+               nativeHandler:(BOOL (^)(NSEvent*))nativePerformKeyEquivalent {
   // Some extension commands have higher priority than web content, and some
   // have lower priority. Regardless of whether the event is being
   // redispatched, let the extension system try to handle the event.
-  NSWindow* window = event.window;
-  if (window) {
-    BrowserWindowController* controller = [window windowController];
+  if (event.window) {

[tapted, 2015/07/31 03:06:49]

event.window -> window? (or, owner_?) (or, comment why not?) 

[jackhou1, 2015/08/04 01:03:58]

Done.

+    BrowserWindowController* controller = [event.window windowController];

[tapted, 2015/07/31 03:06:49]

ObjCCastStrict?

And.. that might actually fail. It looks like NativeAppWindowController
implements handledByExtensionCommand: but doesn't inherit from
BrowserWindowController.

Probably |controller| should just be declared `id`... [or there should be a
separate protocol..]

[tapted, 2015/07/31 03:06:50]

event.window -> window?

[jackhou1, 2015/08/04 01:03:58]

Done.

     if ([controller respondsToSelector:@selector(handledByExtensionCommand:
                                                                   priority:)]) {
       ui::AcceleratorManager::HandlerPriority priority =
           redispatchingEvent_ ? ui::AcceleratorManager::kNormalPriority
                               : ui::AcceleratorManager::kHighPriority;
       if ([controller handledByExtensionCommand:event priority:priority])
         return YES;
     }
   }
 
   if (redispatchingEvent_)
     return NO;
 
   // Give the web site a chance to handle the event. If it doesn't want to
   // handle it, it will call us back with one of the |handle*| methods above.
-  NSResponder* r = [self firstResponder];
+  NSResponder* r = [window firstResponder];
   if ([r conformsToProtocol:@protocol(RenderWidgetHostViewMacBase)])
     return [r performKeyEquivalent:event];
 
   // Handle per-window shortcuts like cmd-1, but do not handle browser-level
   // shortcuts like cmd-left (else, cmd-left would do history navigation even
   // if e.g. the Omnibox has focus).
-  if (HandleExtraWindowKeyboardShortcut(event, self))
+  if (HandleExtraWindowKeyboardShortcut(event, window))
     return YES;
 
-  if ([super performKeyEquivalent:event])
+  if (nativePerformKeyEquivalent(event))

[tapted, 2015/07/31 03:06:50]

e.g. this could become [owner_ defaultPerformKeyEquivalent:event]

[jackhou1, 2015/08/04 01:03:58]

Done.

     return YES;
 
   // Handle per-window shortcuts like Esc after giving everybody else a chance
   // to handle them
-  return HandleDelayedWindowKeyboardShortcut(event, self);
+  return HandleDelayedWindowKeyboardShortcut(event, window);
 }
 
-- (BOOL)redispatchKeyEvent:(NSEvent*)event {
+- (BOOL)redispatchKeyEvent:(NSEvent*)event window:(NSWindow*)window {
   DCHECK(event);
   NSEventType eventType = [event type];
-  if (eventType != NSKeyDown &&
-      eventType != NSKeyUp &&
+  if (eventType != NSKeyDown && eventType != NSKeyUp &&
       eventType != NSFlagsChanged) {
     NOTREACHED();
     return YES;  // Pretend it's been handled in an effort to limit damage.
   }
 
   // Ordinarily, the event's window should be this window. However, when
   // switching between normal and fullscreen mode, we switch out the window, and
   // the event's window might be the previous window (or even an earlier one if
   // the renderer is running slowly and several mode switches occur). In this
   // rare case, we synthesize a new key event so that its associate window
   // (number) is our own.
-  if ([event window] != self)
-    event = KeyEventForWindow(self, event);
+  if ([event window] != window)
+    event = KeyEventForWindow(window, event);
 
   // Redispatch the event.
   eventHandled_ = YES;
   redispatchingEvent_ = YES;
   [NSApp sendEvent:event];
   redispatchingEvent_ = NO;
 
   // If the event was not handled by [NSApp sendEvent:], the sendEvent:
   // method below will be called, and because |redispatchingEvent_| is YES,
   // |eventHandled_| will be set to NO.
   return eventHandled_;
 }
 
-- (void)sendEvent:(NSEvent*)event {
-  if (!redispatchingEvent_)
-    [super sendEvent:event];
-  else
+- (BOOL)preSendEvent:(NSEvent*)event {
+  if (redispatchingEvent_) {
     eventHandled_ = NO;
+    return NO;
+  }
+
+  return YES;
 }
 
-@end  // ChromeEventProcessingWindow
+@end  // ChromeCommandDispatchDelegate