[Mac] Move UI item validation to UserInterfaceItemCommandHandler.

chrome/browser/ui/cocoa/chrome_event_processing_window.mm

 // Copyright (c) 2012 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"
 
 #include "base/logging.h"
 #import "chrome/browser/ui/cocoa/chrome_command_dispatcher_delegate.h"
 
 @implementation ChromeEventProcessingWindow {
@@ skipping 27 matching lines @@
 // CommandDispatchingWindow implementation.
 
 - (BOOL)redispatchKeyEvent:(NSEvent*)event {
   return [commandDispatcher_ redispatchKeyEvent:event];
 }
 
 - (BOOL)defaultPerformKeyEquivalent:(NSEvent*)event {
   return [super performKeyEquivalent:event];
 }
 
+- (BOOL)validateUserInterfaceItem:(id<NSValidatedUserInterfaceItem>)item {

[tapted, 2015/09/03 04:15:38]

move down below NSWindow overrides. Maybe

// NSWindow overrides (NSUserInterfaceValidations implementation).

But then... should it also call [super validateUserInterfaceItem] if the
dispatcher returns YES?

I think it should... I suspect this might be responsible for things like
disabling menu items when a Sheet is open (e.g. Cmd+S)

[jackhou1, 2015/09/03 06:15:39]

Done.

+  return
+      [commandDispatcherDelegate_ validateUserInterfaceItem:item window:self];
+}
+
+- (void)commandDispatch:(id)sender {
+  [commandDispatcherDelegate_ commandDispatch:sender window:self];
+}
+
+- (void)commandDispatchUsingKeyModifiers:(id)sender {
+  [commandDispatcherDelegate_ commandDispatchUsingKeyModifiers:sender
+                                                        window:self];
+}
+
 // NSWindow overrides.
 
 - (BOOL)performKeyEquivalent:(NSEvent*)event {
   return [commandDispatcher_ performKeyEquivalent:event];
 }
 
 - (void)sendEvent:(NSEvent*)event {
   if (![commandDispatcher_ preSendEvent:event])
     [super sendEvent:event];
 }
 
 @end  // ChromeEventProcessingWindow