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

Unified Diff: ui/events/event.cc

Issue 2914023002: Remove LatencyInfo::sequence_number. (May break metrics).
Patch Set: Rebase Created 3 years, 6 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.cc
diff --git a/ui/events/event.cc b/ui/events/event.cc
index 7cb7a32d90f9b7e9942f86bb00bd95c2b1a89c2a..849f36d20d3781d83b74f427176bed829626326e 100644
--- a/ui/events/event.cc
+++ b/ui/events/event.cc
@@ -572,9 +572,9 @@ MouseEvent::MouseEvent(const base::NativeEvent& native_event)
changed_button_flags_(GetChangedMouseButtonFlagsFromNative(native_event)),
pointer_details_(GetMousePointerDetailsFromNative(native_event)) {
latency()->AddLatencyNumberWithTimestamp(
- INPUT_EVENT_LATENCY_ORIGINAL_COMPONENT, 0, 0,
+ INPUT_EVENT_LATENCY_ORIGINAL_COMPONENT, 0,
base::TimeTicks::FromInternalValue(time_stamp().ToInternalValue()), 1);
- latency()->AddLatencyNumber(INPUT_EVENT_LATENCY_UI_COMPONENT, 0, 0);
+ latency()->AddLatencyNumber(INPUT_EVENT_LATENCY_UI_COMPONENT, 0);
if (type() == ET_MOUSE_PRESSED || type() == ET_MOUSE_RELEASED)
SetClickCount(GetRepeatCount(*this));
}
@@ -640,7 +640,7 @@ MouseEvent::MouseEvent(EventType type,
changed_button_flags_(changed_button_flags),
pointer_details_(pointer_details) {
DCHECK_NE(ET_MOUSEWHEEL, type);
- latency()->AddLatencyNumber(INPUT_EVENT_LATENCY_UI_COMPONENT, 0, 0);
+ latency()->AddLatencyNumber(INPUT_EVENT_LATENCY_UI_COMPONENT, 0);
if (this->type() == ET_MOUSE_MOVED && IsAnyButton())
SetType(ET_MOUSE_DRAGGED);
}
@@ -840,9 +840,9 @@ TouchEvent::TouchEvent(const base::NativeEvent& native_event)
should_remove_native_touch_id_mapping_(false),
pointer_details_(GetTouchPointerDetailsFromNative(native_event)) {
latency()->AddLatencyNumberWithTimestamp(
- INPUT_EVENT_LATENCY_ORIGINAL_COMPONENT, 0, 0,
+ INPUT_EVENT_LATENCY_ORIGINAL_COMPONENT, 0,
base::TimeTicks::FromInternalValue(time_stamp().ToInternalValue()), 1);
- latency()->AddLatencyNumber(INPUT_EVENT_LATENCY_UI_COMPONENT, 0, 0);
+ latency()->AddLatencyNumber(INPUT_EVENT_LATENCY_UI_COMPONENT, 0);
FixRotationAngle();
if (type() == ET_TOUCH_RELEASED || type() == ET_TOUCH_CANCELLED)
@@ -895,7 +895,7 @@ TouchEvent::TouchEvent(EventType type,
may_cause_scrolling_(false),
should_remove_native_touch_id_mapping_(false),
pointer_details_(pointer_details) {
- latency()->AddLatencyNumber(INPUT_EVENT_LATENCY_UI_COMPONENT, 0, 0);
+ latency()->AddLatencyNumber(INPUT_EVENT_LATENCY_UI_COMPONENT, 0);
FixRotationAngle();
}
@@ -1149,9 +1149,9 @@ KeyEvent::KeyEvent(const base::NativeEvent& native_event, int event_flags)
code_(CodeFromNative(native_event)),
is_char_(IsCharFromNative(native_event)) {
latency()->AddLatencyNumberWithTimestamp(
- INPUT_EVENT_LATENCY_ORIGINAL_COMPONENT, 0, 0,
+ INPUT_EVENT_LATENCY_ORIGINAL_COMPONENT, 0,
base::TimeTicks::FromInternalValue(time_stamp().ToInternalValue()), 1);
- latency()->AddLatencyNumber(INPUT_EVENT_LATENCY_UI_COMPONENT, 0, 0);
+ latency()->AddLatencyNumber(INPUT_EVENT_LATENCY_UI_COMPONENT, 0);
if (IsRepeated(*this))
set_flags(flags() | ui::EF_IS_REPEAT);

Powered by Google App Engine
This is Rietveld 408576698