MacViews: Views accelerators table should match the Cocoa one.

chrome/browser/ui/views/frame/browser_frame.h

 // 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.
 
 #ifndef CHROME_BROWSER_UI_VIEWS_FRAME_BROWSER_FRAME_H_
 #define CHROME_BROWSER_UI_VIEWS_FRAME_BROWSER_FRAME_H_
 
 #include "base/compiler_specific.h"
 #include "base/logging.h"
 #include "base/macros.h"
 #include "build/build_config.h"
 #include "chrome/browser/ui/views/frame/browser_non_client_frame_view.h"
 #include "ui/views/context_menu_controller.h"
 #include "ui/views/widget/widget.h"
 
 class AvatarMenuButton;
 class BrowserRootView;
 class BrowserView;
 class NativeBrowserFrame;
 class NonClientFrameView;
 class SystemMenuModelBuilder;
 
+namespace content {
+struct NativeWebKeyboardEvent;
+}
+
 namespace gfx {
 class FontList;
 class Rect;
 }
 
 namespace ui {
 class EventHandler;
 class MenuModel;
 }
 
@@ skipping 52 matching lines @@
   // Returns |true| if we should use the custom frame.
   bool UseCustomFrame() const;
 
   // Returns true when the window placement should be saved.
   bool ShouldSaveWindowPlacement() const;
 
   // Retrieves the window placement (show state and bounds) for restoring.
   void GetWindowPlacement(gfx::Rect* bounds,
                           ui::WindowShowState* show_state) const;
 
+  // Returns true if the |event| was handled by the platform implementation
+  // before sending it to the renderer. E.g., it may be swallowed by a native
+  // menu bar.
+  bool PreHandleKeyboardEvent(const content::NativeWebKeyboardEvent& event);

[sky, 2016/11/08 18:34:10]

Please use an enum for the return type to make the return value clear.

+
+  // Returns true if the |event| was handled by the platform implementation,
+  // if the renderer did not process it.
+  bool HandleKeyboardEvent(const content::NativeWebKeyboardEvent& event);
+
   // Called when BrowserView creates all it's child views.
   void OnBrowserViewInitViewsComplete();
 
   // Overridden from views::Widget:
   views::internal::RootView* CreateRootView() override;
   views::NonClientFrameView* CreateNonClientFrameView() override;
   bool GetAccelerator(int command_id,
                       ui::Accelerator* accelerator) const override;
   const ui::ThemeProvider* GetThemeProvider() const override;
   const ui::NativeTheme* GetNativeTheme() const override;
@@ skipping 34 matching lines @@
   // Used to show the system menu. Only used if
   // NativeBrowserFrame::UsesNativeSystemMenu() returns false.
   std::unique_ptr<views::MenuRunner> menu_runner_;
 
   std::unique_ptr<ui::EventHandler> browser_command_handler_;
 
   DISALLOW_COPY_AND_ASSIGN(BrowserFrame);
 };
 
 #endif  // CHROME_BROWSER_UI_VIEWS_FRAME_BROWSER_FRAME_H_