Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2673)

Unified Diff: chrome/browser/ui/views/frame/browser_view_platform.h

Issue 2074643003: MacViews: Views accelerators table should match the Cocoa one. (Closed) Base URL: ssh://bitbucket.browser.yandex-team.ru/chromium/src.git@master
Patch Set: Updated accelerator_table.cc, removed non-Cocoa shortcuts, added global_keyboard_shortcuts_mac.mm h… Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/views/frame/browser_view_platform.h
diff --git a/chrome/browser/ui/views/frame/browser_view_platform.h b/chrome/browser/ui/views/frame/browser_view_platform.h
new file mode 100644
index 0000000000000000000000000000000000000000..8b10e14bf9dbd79dea10d60e19bd0648cd416aed
--- /dev/null
+++ b/chrome/browser/ui/views/frame/browser_view_platform.h
@@ -0,0 +1,28 @@
+// Copyright 2016 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_VIEW_PLATFORM_H_
+#define CHROME_BROWSER_UI_VIEWS_FRAME_BROWSER_VIEW_PLATFORM_H_
+
+#include "content/public/browser/native_web_keyboard_event.h"
+
+class Browser;
+
+// Handles the platform-specific differences that couldn't be implemented
+// directly inside BrowserView.
+class BrowserViewPlatform {
+ public:
+ // On Cocoa we have three additional global shortcut tables that couldn't be
+ // migrated to accelerator_table.cc (crbug.com/25946).
+
+ // Returns true if the |event| was handled by the platform implementation.
+ static bool PreHandleKeyboardEvent(
+ const content::NativeWebKeyboardEvent& event,
+ Browser* browser);
+ // Returns true if the |event| was handled by the platform implementation.
tapted 2016/10/17 07:02:47 nit: blank line before
themblsha 2016/10/20 16:41:05 Done.
+ static bool HandleKeyboardEvent(const content::NativeWebKeyboardEvent& event,
+ Browser* browser);
+};
+
+#endif // CHROME_BROWSER_UI_VIEWS_FRAME_BROWSER_VIEW_PLATFORM_H_

Powered by Google App Engine
This is Rietveld 408576698