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

Unified Diff: ui/events/event_unittest.cc

Issue 2317253005: SourceEventType added to LatencyInfo (Closed)
Patch Set: Added a fake return to the end of EventTypeToLatencySourceEventType, for compilers that don't know … Created 4 years, 3 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/events/event_unittest.cc
diff --git a/ui/events/event_unittest.cc b/ui/events/event_unittest.cc
index ae9a5d6bd059250861531a725bcfac2a6c220f9e..7fe023a144fb0390e40cd8321671578b42c1b367 100644
--- a/ui/events/event_unittest.cc
+++ b/ui/events/event_unittest.cc
@@ -967,6 +967,22 @@ TEST(EventTest, PointerEventToTouchEventDetails) {
EXPECT_EQ(pointer_event.time_stamp(), touch_event.time_stamp());
}
+TEST(EventTest, PointerEventSourceEventTypeExistsInLatencyInfo) {
+ ui::PointerEvent wheel_poniter_event(
+ ui::ET_POINTER_WHEEL_CHANGED, gfx::Point(), gfx::Point(), 0, 0, 0,
+ ui::PointerDetails(ui::EventPointerType::POINTER_TYPE_MOUSE),
+ ui::EventTimeForNow());
+ EXPECT_EQ(wheel_poniter_event.latency()->source_event_type(),
+ ui::SourceEventType::WHEEL);
+
+ ui::PointerEvent touch_poniter_event(
+ ui::ET_TOUCH_PRESSED, gfx::Point(), gfx::Point(), 0, 0, 0,
+ ui::PointerDetails(ui::EventPointerType::POINTER_TYPE_TOUCH),
+ ui::EventTimeForNow());
+ EXPECT_EQ(touch_poniter_event.latency()->source_event_type(),
+ ui::SourceEventType::TOUCH);
+}
+
// Checks that Event.Latency.OS.TOUCH_PRESSED, TOUCH_MOVED,
// and TOUCH_RELEASED histograms are computed properly.
#if defined(USE_X11)

Powered by Google App Engine
This is Rietveld 408576698