diff options
author | sadrul@chromium.org <sadrul@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-12-14 17:16:21 +0000 |
---|---|---|
committer | sadrul@chromium.org <sadrul@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-12-14 17:16:21 +0000 |
commit | 5eebaf46e7ea120921c3fcfce64ea45047659641 (patch) | |
tree | 6afa9a413c557de459ee0ad4c997bd26404ebffe /ash/wm/user_activity_detector_unittest.cc | |
parent | 25cc762f00cbc1904e997f789078162e51e91418 (diff) | |
download | chromium_src-5eebaf46e7ea120921c3fcfce64ea45047659641.zip chromium_src-5eebaf46e7ea120921c3fcfce64ea45047659641.tar.gz chromium_src-5eebaf46e7ea120921c3fcfce64ea45047659641.tar.bz2 |
events: Update key-event handlers to not return EventResult.
BUG=163618
Review URL: https://codereview.chromium.org/11570012
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@173171 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'ash/wm/user_activity_detector_unittest.cc')
-rw-r--r-- | ash/wm/user_activity_detector_unittest.cc | 15 |
1 files changed, 10 insertions, 5 deletions
diff --git a/ash/wm/user_activity_detector_unittest.cc b/ash/wm/user_activity_detector_unittest.cc index 1e3105b..979c9ea 100644 --- a/ash/wm/user_activity_detector_unittest.cc +++ b/ash/wm/user_activity_detector_unittest.cc @@ -92,7 +92,8 @@ TEST_F(UserActivityDetectorTest, Basic) { ui::KeyEvent key_event(ui::ET_KEY_PRESSED, ui::VKEY_A, ui::EF_NONE, false); SetEventTarget(window.get(), &key_event); - EXPECT_EQ(ui::ER_UNHANDLED, detector_->OnKeyEvent(&key_event)); + detector_->OnKeyEvent(&key_event); + EXPECT_FALSE(key_event.handled()); EXPECT_EQ(1, observer_->num_invocations()); observer_->reset_stats(); @@ -141,13 +142,15 @@ TEST_F(UserActivityDetectorTest, RateLimitNotifications) { // The observer should be notified about a key event. ui::KeyEvent event(ui::ET_KEY_PRESSED, ui::VKEY_A, ui::EF_NONE, false); SetEventTarget(window.get(), &event); - EXPECT_FALSE(detector_->OnKeyEvent(&event)); + detector_->OnKeyEvent(&event); + EXPECT_FALSE(event.handled()); EXPECT_EQ(1, observer_->num_invocations()); observer_->reset_stats(); // It shouldn't be notified if a second event occurs // in the same instant in time. - EXPECT_FALSE(detector_->OnKeyEvent(&event)); + detector_->OnKeyEvent(&event); + EXPECT_FALSE(event.handled()); EXPECT_EQ(0, observer_->num_invocations()); observer_->reset_stats(); @@ -155,7 +158,8 @@ TEST_F(UserActivityDetectorTest, RateLimitNotifications) { AdvanceTime( base::TimeDelta::FromMilliseconds( UserActivityDetector::kNotifyIntervalMs - 100)); - EXPECT_EQ(ui::ER_UNHANDLED, detector_->OnKeyEvent(&event)); + detector_->OnKeyEvent(&event); + EXPECT_FALSE(event.handled()); EXPECT_EQ(0, observer_->num_invocations()); observer_->reset_stats(); @@ -164,7 +168,8 @@ TEST_F(UserActivityDetectorTest, RateLimitNotifications) { AdvanceTime(base::TimeDelta::FromMilliseconds( UserActivityDetector::kNotifyIntervalMs)); - EXPECT_EQ(ui::ER_UNHANDLED, detector_->OnKeyEvent(&event)); + detector_->OnKeyEvent(&event); + EXPECT_FALSE(event.handled()); EXPECT_EQ(1, observer_->num_invocations()); } |