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

Unified Diff: ui/base/accelerators/accelerator_manager_unittest.cc

Issue 2751323002: Converts ui::Accelerator::type to an enum (Closed)
Patch Set: remove dcheck Created 3 years, 9 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/base/accelerators/accelerator_manager_unittest.cc
diff --git a/ui/base/accelerators/accelerator_manager_unittest.cc b/ui/base/accelerators/accelerator_manager_unittest.cc
index 50e8841c4d6c27d22213d59bf89eedf8c7a9f314..3cca7da27234d7ad513d312fb5874edf479d2c68 100644
--- a/ui/base/accelerators/accelerator_manager_unittest.cc
+++ b/ui/base/accelerators/accelerator_manager_unittest.cc
@@ -247,9 +247,7 @@ TEST_F(AcceleratorManagerTest, Process) {
EXPECT_EQ(last_count + 1, target.accelerator_pressed_count()) << i;
// The non-registered accelerators are not processed.
- accelerator.set_type(ET_UNKNOWN);
- EXPECT_FALSE(manager_.Process(accelerator)) << i; // different type
- accelerator.set_type(ET_KEY_RELEASED);
+ accelerator.set_key_state(Accelerator::KeyState::RELEASED);
EXPECT_FALSE(manager_.Process(accelerator)) << i; // different type
EXPECT_FALSE(manager_.Process(GetAccelerator(VKEY_UNKNOWN, modifiers)))

Powered by Google App Engine
This is Rietveld 408576698