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

Unified Diff: ui/events/gesture_detection/touch_disposition_gesture_filter.cc

Issue 2869823003: [VSync Queue] Plug touch ack to gesture events and flush vsync queue if necessary (Closed)
Patch Set: Fix MSAN Use-of-uninitialized-value: Initialize GestureEventDetails 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/gesture_detection/touch_disposition_gesture_filter.cc
diff --git a/ui/events/gesture_detection/touch_disposition_gesture_filter.cc b/ui/events/gesture_detection/touch_disposition_gesture_filter.cc
index 6668f0d753bc3f59aa1ea4c0207fd57730e9e680..a23bc35a4e84a421fef3098b2316882cfca9e7e3 100644
--- a/ui/events/gesture_detection/touch_disposition_gesture_filter.cc
+++ b/ui/events/gesture_detection/touch_disposition_gesture_filter.cc
@@ -191,7 +191,9 @@ TouchDispositionGestureFilter::OnGesturePacket(
}
void TouchDispositionGestureFilter::OnTouchEventAck(
- uint32_t unique_touch_event_id, bool event_consumed) {
+ uint32_t unique_touch_event_id,
+ bool event_consumed,
+ bool is_source_touch_event_set_non_blocking) {
// Spurious asynchronous acks should not trigger a crash.
if (IsEmpty() || (Head().empty() && sequences_.size() == 1))
return;
@@ -201,13 +203,13 @@ void TouchDispositionGestureFilter::OnTouchEventAck(
if (!Tail().empty() &&
Tail().back().unique_touch_event_id() == unique_touch_event_id) {
- Tail().back().Ack(event_consumed);
+ Tail().back().Ack(event_consumed, is_source_touch_event_set_non_blocking);
if (sequences_.size() == 1 && Tail().size() == 1)
SendAckedEvents();
} else {
DCHECK(!Head().empty());
DCHECK_EQ(Head().front().unique_touch_event_id(), unique_touch_event_id);
- Head().front().Ack(event_consumed);
+ Head().front().Ack(event_consumed, is_source_touch_event_set_non_blocking);
SendAckedEvents();
}
}

Powered by Google App Engine
This is Rietveld 408576698