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

Unified Diff: ui/views/controls/menu/menu_controller.cc

Issue 168033003: Pass in correct parameter to MenuController::SelectByChar() on Linux Aura (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 10 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: ui/views/controls/menu/menu_controller.cc
diff --git a/ui/views/controls/menu/menu_controller.cc b/ui/views/controls/menu/menu_controller.cc
index 480c5aed0b3ad65d9f9499c3d5136e9d03f86fa2..c3f9d26de6610062469392a66254584cb8d986ee 100644
--- a/ui/views/controls/menu/menu_controller.cc
+++ b/ui/views/controls/menu/menu_controller.cc
@@ -18,6 +18,7 @@
#include "ui/base/l10n/l10n_util.h"
#include "ui/events/event_constants.h"
#include "ui/events/event_utils.h"
+#include "ui/events/keycodes/keyboard_code_conversion.h"
#include "ui/events/keycodes/keyboard_codes.h"
#include "ui/gfx/canvas.h"
#include "ui/gfx/native_widget_types.h"
@@ -1071,7 +1072,9 @@ uint32_t MenuController::Dispatch(const base::NativeEvent& event) {
if (!OnKeyDown(ui::KeyboardCodeFromNative(event)))
return POST_DISPATCH_QUIT_LOOP;
- bool should_exit = SelectByChar(ui::KeyboardCodeFromNative(event));
+ char c = ui::GetCharacterFromKeyCode(
+ ui::KeyboardCodeFromNative(event), flags);
+ bool should_exit = SelectByChar(c);
return should_exit ? POST_DISPATCH_QUIT_LOOP : POST_DISPATCH_NONE;
}
case ui::ET_KEY_RELEASED:
« chrome/browser/ui/views/menu_controller_test.cc ('K') | « chrome/chrome_tests.gypi ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698