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

Unified Diff: ui/events/blink/blink_event_util.cc

Issue 2746513002: [Compositor event queue] Queue GestureFlingStart/Cancel together with scroll/pinch (Closed)
Patch Set: 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/events/blink/blink_event_util.cc
diff --git a/ui/events/blink/blink_event_util.cc b/ui/events/blink/blink_event_util.cc
index f382fc1ce5bbc1f5519eed6dcbe3a358a0180828..20e0d3bde8982837a2f82d980dcf1228af453970 100644
--- a/ui/events/blink/blink_event_util.cc
+++ b/ui/events/blink/blink_event_util.cc
@@ -962,11 +962,13 @@ blink::WebInputEvent::Modifiers DomCodeToWebInputEventModifiers(DomCode code) {
return static_cast<blink::WebInputEvent::Modifiers>(0);
}
-bool IsGestureScollOrPinch(WebInputEvent::Type type) {
+bool IsGestureScrollFlingPinch(WebInputEvent::Type type) {
tdresser 2017/03/10 15:31:06 I'd prefer we had an "Or" in there. IsGestureScro
chongz 2017/03/14 14:51:06 Done.
switch (type) {
case blink::WebGestureEvent::GestureScrollBegin:
case blink::WebGestureEvent::GestureScrollUpdate:
case blink::WebGestureEvent::GestureScrollEnd:
+ case blink::WebGestureEvent::GestureFlingStart:
+ case blink::WebGestureEvent::GestureFlingCancel:
case blink::WebGestureEvent::GesturePinchBegin:
case blink::WebGestureEvent::GesturePinchUpdate:
case blink::WebGestureEvent::GesturePinchEnd:

Powered by Google App Engine
This is Rietveld 408576698