| Index: ash/accelerators/accelerator_controller_unittest.cc
|
| diff --git a/ash/accelerators/accelerator_controller_unittest.cc b/ash/accelerators/accelerator_controller_unittest.cc
|
| index 54b6f827d4274e2e024a7e53dcafe1c50d7899f8..463e59e277022237f90490917205b1fadd302d76 100644
|
| --- a/ash/accelerators/accelerator_controller_unittest.cc
|
| +++ b/ash/accelerators/accelerator_controller_unittest.cc
|
| @@ -51,14 +51,6 @@ class TestTarget : public ui::AcceleratorTarget {
|
| DISALLOW_COPY_AND_ASSIGN(TestTarget);
|
| };
|
|
|
| -class ReleaseAccelerator : public ui::Accelerator {
|
| - public:
|
| - ReleaseAccelerator(ui::KeyboardCode keycode, int modifiers)
|
| - : ui::Accelerator(keycode, modifiers) {
|
| - set_type(ui::ET_KEY_RELEASED);
|
| - }
|
| -};
|
| -
|
| class DummyScreenshotDelegate : public ScreenshotDelegate {
|
| public:
|
| DummyScreenshotDelegate()
|
| @@ -284,7 +276,9 @@ AcceleratorController* AcceleratorControllerTest::GetController() {
|
| }
|
|
|
| TEST_F(AcceleratorControllerTest, Register) {
|
| - const ui::Accelerator accelerator_a(ui::VKEY_A, ui::EF_NONE);
|
| + const ui::Accelerator accelerator_a(ui::VKEY_A,
|
| + ui::EF_NONE,
|
| + ui::ET_KEY_PRESSED);
|
| TestTarget target;
|
| GetController()->Register(accelerator_a, &target);
|
|
|
| @@ -294,7 +288,9 @@ TEST_F(AcceleratorControllerTest, Register) {
|
| }
|
|
|
| TEST_F(AcceleratorControllerTest, RegisterMultipleTarget) {
|
| - const ui::Accelerator accelerator_a(ui::VKEY_A, ui::EF_NONE);
|
| + const ui::Accelerator accelerator_a(ui::VKEY_A,
|
| + ui::EF_NONE,
|
| + ui::ET_KEY_PRESSED);
|
| TestTarget target1;
|
| GetController()->Register(accelerator_a, &target1);
|
| TestTarget target2;
|
| @@ -308,10 +304,14 @@ TEST_F(AcceleratorControllerTest, RegisterMultipleTarget) {
|
| }
|
|
|
| TEST_F(AcceleratorControllerTest, Unregister) {
|
| - const ui::Accelerator accelerator_a(ui::VKEY_A, ui::EF_NONE);
|
| + const ui::Accelerator accelerator_a(ui::VKEY_A,
|
| + ui::EF_NONE,
|
| + ui::ET_KEY_PRESSED);
|
| TestTarget target;
|
| GetController()->Register(accelerator_a, &target);
|
| - const ui::Accelerator accelerator_b(ui::VKEY_B, ui::EF_NONE);
|
| + const ui::Accelerator accelerator_b(ui::VKEY_B,
|
| + ui::EF_NONE,
|
| + ui::ET_KEY_PRESSED);
|
| GetController()->Register(accelerator_b, &target);
|
|
|
| // Unregistering a different accelerator does not affect the other
|
| @@ -328,12 +328,18 @@ TEST_F(AcceleratorControllerTest, Unregister) {
|
| }
|
|
|
| TEST_F(AcceleratorControllerTest, UnregisterAll) {
|
| - const ui::Accelerator accelerator_a(ui::VKEY_A, ui::EF_NONE);
|
| + const ui::Accelerator accelerator_a(ui::VKEY_A,
|
| + ui::EF_NONE,
|
| + ui::ET_KEY_PRESSED);
|
| TestTarget target1;
|
| GetController()->Register(accelerator_a, &target1);
|
| - const ui::Accelerator accelerator_b(ui::VKEY_B, ui::EF_NONE);
|
| + const ui::Accelerator accelerator_b(ui::VKEY_B,
|
| + ui::EF_NONE,
|
| + ui::ET_KEY_PRESSED);
|
| GetController()->Register(accelerator_b, &target1);
|
| - const ui::Accelerator accelerator_c(ui::VKEY_C, ui::EF_NONE);
|
| + const ui::Accelerator accelerator_c(ui::VKEY_C,
|
| + ui::EF_NONE,
|
| + ui::ET_KEY_PRESSED);
|
| TestTarget target2;
|
| GetController()->Register(accelerator_c, &target2);
|
| GetController()->UnregisterAll(&target1);
|
| @@ -349,7 +355,9 @@ TEST_F(AcceleratorControllerTest, UnregisterAll) {
|
| }
|
|
|
| TEST_F(AcceleratorControllerTest, Process) {
|
| - const ui::Accelerator accelerator_a(ui::VKEY_A, ui::EF_NONE);
|
| + const ui::Accelerator accelerator_a(ui::VKEY_A,
|
| + ui::EF_NONE,
|
| + ui::ET_KEY_PRESSED);
|
| TestTarget target1;
|
| GetController()->Register(accelerator_a, &target1);
|
|
|
| @@ -358,13 +366,19 @@ TEST_F(AcceleratorControllerTest, Process) {
|
| EXPECT_EQ(1, target1.accelerator_pressed_count());
|
|
|
| // The non-registered accelerator is not processed.
|
| - const ui::Accelerator accelerator_b(ui::VKEY_B, ui::EF_NONE);
|
| + const ui::Accelerator accelerator_b(ui::VKEY_B,
|
| + ui::EF_NONE,
|
| + ui::ET_KEY_PRESSED);
|
| EXPECT_FALSE(GetController()->Process(accelerator_b));
|
| }
|
|
|
| TEST_F(AcceleratorControllerTest, IsRegistered) {
|
| - const ui::Accelerator accelerator_a(ui::VKEY_A, ui::EF_NONE);
|
| - const ui::Accelerator accelerator_shift_a(ui::VKEY_A, ui::EF_SHIFT_DOWN);
|
| + const ui::Accelerator accelerator_a(ui::VKEY_A,
|
| + ui::EF_NONE,
|
| + ui::ET_KEY_PRESSED);
|
| + const ui::Accelerator accelerator_shift_a(ui::VKEY_A,
|
| + ui::EF_SHIFT_DOWN,
|
| + ui::ET_KEY_PRESSED);
|
| TestTarget target;
|
| GetController()->Register(accelerator_a, &target);
|
| EXPECT_TRUE(GetController()->IsRegistered(accelerator_a));
|
| @@ -375,7 +389,9 @@ TEST_F(AcceleratorControllerTest, IsRegistered) {
|
|
|
| #if defined(OS_WIN) || defined(USE_X11)
|
| TEST_F(AcceleratorControllerTest, ProcessOnce) {
|
| - ui::Accelerator accelerator_a(ui::VKEY_A, ui::EF_NONE);
|
| + ui::Accelerator accelerator_a(ui::VKEY_A,
|
| + ui::EF_NONE,
|
| + ui::ET_KEY_PRESSED);
|
| TestTarget target;
|
| GetController()->Register(accelerator_a, &target);
|
|
|
| @@ -419,38 +435,44 @@ TEST_F(AcceleratorControllerTest, ProcessOnce) {
|
| TEST_F(AcceleratorControllerTest, DISABLED_GlobalAccelerators) {
|
| // CycleBackward
|
| EXPECT_TRUE(GetController()->Process(
|
| - ui::Accelerator(ui::VKEY_F5, ui::EF_CONTROL_DOWN)));
|
| + ui::Accelerator(ui::VKEY_F5, ui::EF_CONTROL_DOWN, ui::ET_KEY_PRESSED)));
|
| EXPECT_TRUE(GetController()->Process(
|
| - ui::Accelerator(ui::VKEY_TAB, ui::EF_SHIFT_DOWN | ui::EF_ALT_DOWN)));
|
| + ui::Accelerator(ui::VKEY_TAB,
|
| + ui::EF_SHIFT_DOWN | ui::EF_ALT_DOWN,
|
| + ui::ET_KEY_PRESSED)));
|
| // CycleForward
|
| EXPECT_TRUE(GetController()->Process(
|
| - ui::Accelerator(ui::VKEY_F5, ui::EF_NONE)));
|
| + ui::Accelerator(ui::VKEY_F5, ui::EF_NONE, ui::ET_KEY_PRESSED)));
|
| EXPECT_TRUE(GetController()->Process(
|
| - ui::Accelerator(ui::VKEY_TAB, ui::EF_ALT_DOWN)));
|
| + ui::Accelerator(ui::VKEY_TAB, ui::EF_ALT_DOWN, ui::ET_KEY_PRESSED)));
|
| // Take screenshot / partial screenshot
|
| // True should always be returned regardless of the existence of the delegate.
|
| {
|
| EXPECT_TRUE(GetController()->Process(
|
| - ui::Accelerator(ui::VKEY_F5, ui::EF_CONTROL_DOWN)));
|
| + ui::Accelerator(ui::VKEY_F5, ui::EF_CONTROL_DOWN, ui::ET_KEY_PRESSED)));
|
| EXPECT_TRUE(GetController()->Process(
|
| - ui::Accelerator(ui::VKEY_PRINT, ui::EF_NONE)));
|
| + ui::Accelerator(ui::VKEY_PRINT, ui::EF_NONE, ui::ET_KEY_PRESSED)));
|
| EXPECT_TRUE(GetController()->Process(
|
| - ui::Accelerator(ui::VKEY_F5, ui::EF_SHIFT_DOWN | ui::EF_CONTROL_DOWN)));
|
| + ui::Accelerator(ui::VKEY_F5,
|
| + ui::EF_SHIFT_DOWN | ui::EF_CONTROL_DOWN,
|
| + ui::ET_KEY_PRESSED)));
|
| DummyScreenshotDelegate* delegate = new DummyScreenshotDelegate;
|
| GetController()->SetScreenshotDelegate(
|
| scoped_ptr<ScreenshotDelegate>(delegate).Pass());
|
| EXPECT_EQ(0, delegate->handle_take_screenshot_count());
|
| EXPECT_EQ(0, delegate->handle_take_partial_screenshot_count());
|
| EXPECT_TRUE(GetController()->Process(
|
| - ui::Accelerator(ui::VKEY_F5, ui::EF_CONTROL_DOWN)));
|
| + ui::Accelerator(ui::VKEY_F5, ui::EF_CONTROL_DOWN, ui::ET_KEY_PRESSED)));
|
| EXPECT_EQ(1, delegate->handle_take_screenshot_count());
|
| EXPECT_EQ(0, delegate->handle_take_partial_screenshot_count());
|
| EXPECT_TRUE(GetController()->Process(
|
| - ui::Accelerator(ui::VKEY_PRINT, ui::EF_NONE)));
|
| + ui::Accelerator(ui::VKEY_PRINT, ui::EF_NONE, ui::ET_KEY_PRESSED)));
|
| EXPECT_EQ(2, delegate->handle_take_screenshot_count());
|
| EXPECT_EQ(0, delegate->handle_take_partial_screenshot_count());
|
| EXPECT_TRUE(GetController()->Process(
|
| - ui::Accelerator(ui::VKEY_F5, ui::EF_SHIFT_DOWN | ui::EF_CONTROL_DOWN)));
|
| + ui::Accelerator(ui::VKEY_F5,
|
| + ui::EF_SHIFT_DOWN | ui::EF_CONTROL_DOWN,
|
| + ui::ET_KEY_PRESSED)));
|
| EXPECT_EQ(2, delegate->handle_take_screenshot_count());
|
| EXPECT_EQ(1, delegate->handle_take_partial_screenshot_count());
|
| }
|
| @@ -458,22 +480,26 @@ TEST_F(AcceleratorControllerTest, DISABLED_GlobalAccelerators) {
|
| {
|
| EXPECT_FALSE(ash::Shell::GetInstance()->GetAppListTargetVisibility());
|
| EXPECT_TRUE(GetController()->Process(
|
| - ui::Accelerator(ui::VKEY_LWIN, ui::EF_CONTROL_DOWN)));
|
| + ui::Accelerator(ui::VKEY_LWIN,
|
| + ui::EF_CONTROL_DOWN,
|
| + ui::ET_KEY_PRESSED)));
|
| EXPECT_TRUE(ash::Shell::GetInstance()->GetAppListTargetVisibility());
|
| EXPECT_TRUE(GetController()->Process(
|
| - ui::Accelerator(ui::VKEY_LWIN, ui::EF_CONTROL_DOWN)));
|
| + ui::Accelerator(ui::VKEY_LWIN,
|
| + ui::EF_CONTROL_DOWN,
|
| + ui::ET_KEY_PRESSED)));
|
| EXPECT_FALSE(ash::Shell::GetInstance()->GetAppListTargetVisibility());
|
| }
|
| // ToggleCapsLock
|
| {
|
| EXPECT_FALSE(GetController()->Process(
|
| - ui::Accelerator(ui::VKEY_LWIN, ui::EF_SHIFT_DOWN)));
|
| + ui::Accelerator(ui::VKEY_LWIN, ui::EF_SHIFT_DOWN, ui::ET_KEY_PRESSED)));
|
| DummyCapsLockDelegate* delegate = new DummyCapsLockDelegate(false);
|
| GetController()->SetCapsLockDelegate(
|
| scoped_ptr<CapsLockDelegate>(delegate).Pass());
|
| EXPECT_EQ(0, delegate->handle_caps_lock_count());
|
| EXPECT_FALSE(GetController()->Process(
|
| - ui::Accelerator(ui::VKEY_LWIN, ui::EF_SHIFT_DOWN)));
|
| + ui::Accelerator(ui::VKEY_LWIN, ui::EF_SHIFT_DOWN, ui::ET_KEY_PRESSED)));
|
| EXPECT_EQ(1, delegate->handle_caps_lock_count());
|
| }
|
| {
|
| @@ -482,13 +508,13 @@ TEST_F(AcceleratorControllerTest, DISABLED_GlobalAccelerators) {
|
| scoped_ptr<CapsLockDelegate>(delegate).Pass());
|
| EXPECT_EQ(0, delegate->handle_caps_lock_count());
|
| EXPECT_TRUE(GetController()->Process(
|
| - ui::Accelerator(ui::VKEY_LWIN, ui::EF_SHIFT_DOWN)));
|
| + ui::Accelerator(ui::VKEY_LWIN, ui::EF_SHIFT_DOWN, ui::ET_KEY_PRESSED)));
|
| EXPECT_EQ(1, delegate->handle_caps_lock_count());
|
| }
|
| // Volume
|
| - const ui::Accelerator f8(ui::VKEY_F8, ui::EF_NONE);
|
| - const ui::Accelerator f9(ui::VKEY_F9, ui::EF_NONE);
|
| - const ui::Accelerator f10(ui::VKEY_F10, ui::EF_NONE);
|
| + const ui::Accelerator f8(ui::VKEY_F8, ui::EF_NONE, ui::ET_KEY_PRESSED);
|
| + const ui::Accelerator f9(ui::VKEY_F9, ui::EF_NONE, ui::ET_KEY_PRESSED);
|
| + const ui::Accelerator f10(ui::VKEY_F10, ui::EF_NONE, ui::ET_KEY_PRESSED);
|
| {
|
| EXPECT_FALSE(GetController()->Process(f8));
|
| EXPECT_FALSE(GetController()->Process(f9));
|
| @@ -527,9 +553,15 @@ TEST_F(AcceleratorControllerTest, DISABLED_GlobalAccelerators) {
|
| EXPECT_EQ(1, delegate->handle_volume_up_count());
|
| EXPECT_EQ(f10, delegate->last_accelerator());
|
| }
|
| - const ui::Accelerator volume_mute(ui::VKEY_VOLUME_MUTE, ui::EF_NONE);
|
| - const ui::Accelerator volume_down(ui::VKEY_VOLUME_DOWN, ui::EF_NONE);
|
| - const ui::Accelerator volume_up(ui::VKEY_VOLUME_UP, ui::EF_NONE);
|
| + const ui::Accelerator volume_mute(ui::VKEY_VOLUME_MUTE,
|
| + ui::EF_NONE,
|
| + ui::ET_KEY_PRESSED);
|
| + const ui::Accelerator volume_down(ui::VKEY_VOLUME_DOWN,
|
| + ui::EF_NONE,
|
| + ui::ET_KEY_PRESSED);
|
| + const ui::Accelerator volume_up(ui::VKEY_VOLUME_UP,
|
| + ui::EF_NONE,
|
| + ui::ET_KEY_PRESSED);
|
| {
|
| DummyVolumeControlDelegate* delegate =
|
| new DummyVolumeControlDelegate(false);
|
| @@ -566,8 +598,8 @@ TEST_F(AcceleratorControllerTest, DISABLED_GlobalAccelerators) {
|
| EXPECT_EQ(volume_up, delegate->last_accelerator());
|
| }
|
| // Brightness
|
| - const ui::Accelerator f6(ui::VKEY_F6, ui::EF_NONE);
|
| - const ui::Accelerator f7(ui::VKEY_F7, ui::EF_NONE);
|
| + const ui::Accelerator f6(ui::VKEY_F6, ui::EF_NONE, ui::ET_KEY_PRESSED);
|
| + const ui::Accelerator f7(ui::VKEY_F7, ui::EF_NONE, ui::ET_KEY_PRESSED);
|
| {
|
| EXPECT_FALSE(GetController()->Process(f6));
|
| EXPECT_FALSE(GetController()->Process(f7));
|
| @@ -600,8 +632,12 @@ TEST_F(AcceleratorControllerTest, DISABLED_GlobalAccelerators) {
|
| }
|
| #if defined(OS_CHROMEOS)
|
| // ui::VKEY_BRIGHTNESS_DOWN/UP are not defined on Windows.
|
| - const ui::Accelerator brightness_down(ui::VKEY_BRIGHTNESS_DOWN, ui::EF_NONE);
|
| - const ui::Accelerator brightness_up(ui::VKEY_BRIGHTNESS_UP, ui::EF_NONE);
|
| + const ui::Accelerator brightness_down(ui::VKEY_BRIGHTNESS_DOWN,
|
| + ui::EF_NONE,
|
| + ui::ET_KEY_PRESSED);
|
| + const ui::Accelerator brightness_up(ui::VKEY_BRIGHTNESS_UP,
|
| + ui::EF_NONE,
|
| + ui::ET_KEY_PRESSED);
|
| {
|
| DummyBrightnessControlDelegate* delegate =
|
| new DummyBrightnessControlDelegate(false);
|
| @@ -634,45 +670,69 @@ TEST_F(AcceleratorControllerTest, DISABLED_GlobalAccelerators) {
|
| #if !defined(NDEBUG)
|
| // RotateScreen
|
| EXPECT_TRUE(GetController()->Process(
|
| - ui::Accelerator(ui::VKEY_HOME, ui::EF_CONTROL_DOWN)));
|
| + ui::Accelerator(ui::VKEY_HOME, ui::EF_CONTROL_DOWN, ui::ET_KEY_PRESSED)));
|
| // ToggleDesktopBackgroundMode
|
| EXPECT_TRUE(GetController()->Process(
|
| - ui::Accelerator(ui::VKEY_B, ui::EF_CONTROL_DOWN | ui::EF_ALT_DOWN)));
|
| + ui::Accelerator(ui::VKEY_B,
|
| + ui::EF_CONTROL_DOWN | ui::EF_ALT_DOWN,
|
| + ui::ET_KEY_PRESSED)));
|
| #if !defined(OS_LINUX)
|
| // ToggleDesktopFullScreen (not implemented yet on Linux)
|
| EXPECT_TRUE(GetController()->Process(
|
| - ui::Accelerator(ui::VKEY_F11, ui::EF_CONTROL_DOWN)));
|
| + ui::Accelerator(ui::VKEY_F11, ui::EF_CONTROL_DOWN, ui::ET_KEY_PRESSED)));
|
| #endif // OS_LINUX
|
| #endif // !NDEBUG
|
|
|
| // Exit
|
| EXPECT_TRUE(GetController()->Process(
|
| - ui::Accelerator(ui::VKEY_Q, ui::EF_SHIFT_DOWN | ui::EF_CONTROL_DOWN)));
|
| + ui::Accelerator(ui::VKEY_Q,
|
| + ui::EF_SHIFT_DOWN | ui::EF_CONTROL_DOWN,
|
| + ui::ET_KEY_PRESSED)));
|
|
|
| // New incognito window
|
| EXPECT_TRUE(GetController()->Process(
|
| - ui::Accelerator(ui::VKEY_N, ui::EF_SHIFT_DOWN | ui::EF_CONTROL_DOWN)));
|
| + ui::Accelerator(ui::VKEY_N,
|
| + ui::EF_SHIFT_DOWN | ui::EF_CONTROL_DOWN,
|
| + ui::ET_KEY_PRESSED)));
|
|
|
| // New window
|
| EXPECT_TRUE(GetController()->Process(
|
| - ui::Accelerator(ui::VKEY_N, ui::EF_CONTROL_DOWN)));
|
| + ui::Accelerator(ui::VKEY_N,
|
| + ui::EF_CONTROL_DOWN,
|
| + ui::ET_KEY_PRESSED)));
|
|
|
| #if defined(OS_CHROMEOS)
|
| EXPECT_TRUE(GetController()->Process(
|
| - ui::Accelerator(ui::VKEY_L, ui::EF_SHIFT_DOWN | ui::EF_CONTROL_DOWN)));
|
| + ui::Accelerator(ui::VKEY_L,
|
| + ui::EF_SHIFT_DOWN | ui::EF_CONTROL_DOWN,
|
| + ui::ET_KEY_PRESSED)));
|
| #endif
|
| }
|
|
|
| TEST_F(AcceleratorControllerTest, ImeGlobalAccelerators) {
|
| // Test IME shortcuts.
|
| {
|
| - const ui::Accelerator control_space(ui::VKEY_SPACE, ui::EF_CONTROL_DOWN);
|
| - const ui::Accelerator convert(ui::VKEY_CONVERT, ui::EF_NONE);
|
| - const ui::Accelerator non_convert(ui::VKEY_NONCONVERT, ui::EF_NONE);
|
| - const ui::Accelerator wide_half_1(ui::VKEY_DBE_SBCSCHAR, ui::EF_NONE);
|
| - const ui::Accelerator wide_half_2(ui::VKEY_DBE_DBCSCHAR, ui::EF_NONE);
|
| - const ui::Accelerator hangul(ui::VKEY_HANGUL, ui::EF_NONE);
|
| - const ui::Accelerator shift_space(ui::VKEY_SPACE, ui::EF_SHIFT_DOWN);
|
| + const ui::Accelerator control_space(ui::VKEY_SPACE,
|
| + ui::EF_CONTROL_DOWN,
|
| + ui::ET_KEY_PRESSED);
|
| + const ui::Accelerator convert(ui::VKEY_CONVERT,
|
| + ui::EF_NONE,
|
| + ui::ET_KEY_PRESSED);
|
| + const ui::Accelerator non_convert(ui::VKEY_NONCONVERT,
|
| + ui::EF_NONE,
|
| + ui::ET_KEY_PRESSED);
|
| + const ui::Accelerator wide_half_1(ui::VKEY_DBE_SBCSCHAR,
|
| + ui::EF_NONE,
|
| + ui::ET_KEY_PRESSED);
|
| + const ui::Accelerator wide_half_2(ui::VKEY_DBE_DBCSCHAR,
|
| + ui::EF_NONE,
|
| + ui::ET_KEY_PRESSED);
|
| + const ui::Accelerator hangul(ui::VKEY_HANGUL,
|
| + ui::EF_NONE,
|
| + ui::ET_KEY_PRESSED);
|
| + const ui::Accelerator shift_space(ui::VKEY_SPACE,
|
| + ui::EF_SHIFT_DOWN,
|
| + ui::ET_KEY_PRESSED);
|
| EXPECT_FALSE(GetController()->Process(control_space));
|
| EXPECT_FALSE(GetController()->Process(convert));
|
| EXPECT_FALSE(GetController()->Process(non_convert));
|
| @@ -704,13 +764,17 @@ TEST_F(AcceleratorControllerTest, ImeGlobalAccelerators) {
|
| // Test IME shortcuts that are triggered on key release.
|
| {
|
| const ui::Accelerator shift_alt_press(ui::VKEY_MENU,
|
| - ui::EF_SHIFT_DOWN | ui::EF_ALT_DOWN);
|
| - const ReleaseAccelerator shift_alt(ui::VKEY_MENU,
|
| - ui::EF_SHIFT_DOWN | ui::EF_ALT_DOWN);
|
| + ui::EF_SHIFT_DOWN | ui::EF_ALT_DOWN,
|
| + ui::ET_KEY_PRESSED);
|
| + const ui::Accelerator shift_alt(ui::VKEY_MENU,
|
| + ui::EF_SHIFT_DOWN | ui::EF_ALT_DOWN,
|
| + ui::ET_KEY_RELEASED);
|
| const ui::Accelerator alt_shift_press(ui::VKEY_SHIFT,
|
| - ui::EF_SHIFT_DOWN | ui::EF_ALT_DOWN);
|
| - const ReleaseAccelerator alt_shift(ui::VKEY_SHIFT,
|
| - ui::EF_SHIFT_DOWN | ui::EF_ALT_DOWN);
|
| + ui::EF_SHIFT_DOWN | ui::EF_ALT_DOWN,
|
| + ui::ET_KEY_PRESSED);
|
| + const ui::Accelerator alt_shift(ui::VKEY_SHIFT,
|
| + ui::EF_SHIFT_DOWN | ui::EF_ALT_DOWN,
|
| + ui::ET_KEY_RELEASED);
|
|
|
| DummyImeControlDelegate* delegate = new DummyImeControlDelegate(true);
|
| GetController()->SetImeControlDelegate(
|
| @@ -725,10 +789,13 @@ TEST_F(AcceleratorControllerTest, ImeGlobalAccelerators) {
|
|
|
| // We should NOT switch IME when e.g. Shift+Alt+X is pressed and X is
|
| // released.
|
| - const ui::Accelerator shift_alt_x_press(ui::VKEY_X,
|
| - ui::EF_SHIFT_DOWN | ui::EF_ALT_DOWN);
|
| - const ReleaseAccelerator shift_alt_x(ui::VKEY_X,
|
| - ui::EF_SHIFT_DOWN | ui::EF_ALT_DOWN);
|
| + const ui::Accelerator shift_alt_x_press(
|
| + ui::VKEY_X,
|
| + ui::EF_SHIFT_DOWN | ui::EF_ALT_DOWN,
|
| + ui::ET_KEY_PRESSED);
|
| + const ui::Accelerator shift_alt_x(ui::VKEY_X,
|
| + ui::EF_SHIFT_DOWN | ui::EF_ALT_DOWN,
|
| + ui::ET_KEY_RELEASED);
|
|
|
| EXPECT_FALSE(GetController()->Process(shift_alt_press));
|
| EXPECT_FALSE(GetController()->Process(shift_alt_x_press));
|
|
|