Pass in correct parameter to MenuController::SelectByChar() on Linux Aura

chrome/browser/ui/views/menu_controller_test.cc

Index: chrome/browser/ui/views/menu_controller_test.cc
diff --git a/chrome/browser/ui/views/menu_controller_test.cc b/chrome/browser/ui/views/menu_controller_test.cc
new file mode 100644
index 0000000000000000000000000000000000000000..3d85b8487e255770caf85e188cfbdd0a6de1c5d3
--- /dev/null
+++ b/chrome/browser/ui/views/menu_controller_test.cc
@@ -0,0 +1,67 @@
+// Copyright 2014 The Chromium Authors. All rights reserved.

[sky, 2014/02/18 17:43:28]

Please name this _browsertest.

+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "base/strings/utf_string_conversions.h"
+#include "chrome/browser/ui/views/menu_test_base.h"
+#include "ui/views/controls/menu/menu_item_view.h"
+#include "ui/views/controls/menu/submenu_view.h"
+
+template<ui::KeyboardCode KEYCODE, int EXPECTED_COMMAND>
+class MenuControllerMnemonicTest : public MenuTestBase {
+ public:
+  MenuControllerMnemonicTest() {
+  }
+
+  virtual ~MenuControllerMnemonicTest() {
+  }
+
+  // MenuTestBase overrides:
+  virtual void BuildMenu(views::MenuItemView* menu) OVERRIDE {
+    ASSERT_NE(ui::VKEY_DIVIDE, '/');
+    menu->AppendMenuItemWithLabel(1, base::ASCIIToUTF16("One&/"));
+    menu->AppendMenuItemWithLabel(2, base::ASCIIToUTF16("Two"));
+  }
+
+  virtual void DoTestWithMenuOpen() {
+    ASSERT_TRUE(menu()->GetSubmenu()->IsShowing());
+    KeyPress(KEYCODE,
+             CreateEventTask(this, &MenuControllerMnemonicTest::Step2));
+  }
+
+  void Step2() {
+    ASSERT_EQ(EXPECTED_COMMAND, last_command());
+    if (EXPECTED_COMMAND == 0) {
+      KeyPress(ui::VKEY_ESCAPE,
+               CreateEventTask(this, &MenuControllerMnemonicTest::Step3));
+    } else {
+      ASSERT_FALSE(menu()->GetSubmenu()->IsShowing());
+      Done();
+    }
+  }
+
+  void Step3() {
+    ASSERT_FALSE(menu()->GetSubmenu()->IsShowing());
+    Done();
+  }
+
+ private:
+  DISALLOW_COPY_AND_ASSIGN(MenuControllerMnemonicTest);
+};
+
+// Pressing the mnemonic for a menu item should execute the command for that
+// menu item.
+typedef MenuControllerMnemonicTest<ui::VKEY_DIVIDE,1>

[sky, 2014/02/18 17:43:28]

nit: space after ',' here and other places.

+    MenuControllerMnemonicTestMnemonicMatch;
+VIEW_TEST(MenuControllerMnemonicTestMnemonicMatch, MnemonicMatch);
+
+// Pressing a key which matches the first letter of the menu item's title
+// should execute the command for that menu item.
+typedef MenuControllerMnemonicTest<ui::VKEY_T,2>
+    MenuControllerMnemonicTestTitleMatch;
+VIEW_TEST(MenuControllerMnemonicTestTitleMatch, TitleMatch);
+
+// Pressing an arbitrary key should not execute any commands.
+typedef MenuControllerMnemonicTest<ui::VKEY_A,0>
+    MenuControllerMnemonicTestNoMatch;
+VIEW_TEST(MenuControllerMnemonicTestNoMatch, NoMatch);