[Mac] Move UI item validation to UserInterfaceItemCommandHandler.

chrome/browser/ui/views/frame/browser_frame_mac.mm

 // Copyright 2014 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 "chrome/browser/ui/views/frame/browser_frame_mac.h"
 
 #import "chrome/browser/ui/cocoa/chrome_command_dispatcher_delegate.h"
 #include "chrome/browser/ui/views/frame/browser_frame.h"
 #include "chrome/browser/ui/views/frame/browser_shutdown.h"
 #include "chrome/browser/ui/views/frame/browser_view.h"
@@ skipping 52 matching lines @@
   NSUInteger style_mask = NSTitledWindowMask | NSClosableWindowMask |
                           NSMiniaturizableWindowMask | NSResizableWindowMask |
                           NSTexturedBackgroundWindowMask;
   base::scoped_nsobject<NativeWidgetMacFramelessNSWindow> ns_window(
       [[NativeWidgetMacFramelessNSWindow alloc]
           initWithContentRect:ui::kWindowSizeDeterminedLater
                     styleMask:style_mask
                       backing:NSBackingStoreBuffered
                         defer:NO]);
   [ns_window setCommandDispatcherDelegate:command_dispatcher_delegate_];
+  [ns_window setUserInterfaceItemCommandHandler:command_dispatcher_delegate_];

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

can NativeWidgetMacNSWindow assume these are always the same for now? (i.e. do
we lose anything?)

Nothing stood out in chrome_command_dispatcher_delegate.mm that would make sense
to handle differently, e.g., for app windows.

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

Done.

   return ns_window.autorelease();
 }
 
 ////////////////////////////////////////////////////////////////////////////////
 // BrowserFrameMac, NativeBrowserFrame implementation:
 
 views::Widget::InitParams BrowserFrameMac::GetWidgetParams() {
   views::Widget::InitParams params;
   params.native_widget = this;
   return params;
@@ skipping 14 matching lines @@
 void BrowserFrameMac::GetWindowPlacement(
     gfx::Rect* bounds,
     ui::WindowShowState* show_state) const {
   return NativeWidgetMac::GetWindowPlacement(bounds, show_state);
 }
 
 int BrowserFrameMac::GetMinimizeButtonOffset() const {
   NOTIMPLEMENTED();
   return 0;
 }