[Compositor event queue] Queue GestureFlingStart/Cancel together with scroll/pinch

ui/events/blink/blink_event_util.h

Index: ui/events/blink/blink_event_util.h
diff --git a/ui/events/blink/blink_event_util.h b/ui/events/blink/blink_event_util.h
index 186428e6c295ada2c7b87bbf247197285d033836..6005649d05d346427a101e4bd93e8e7744d46445 100644
--- a/ui/events/blink/blink_event_util.h
+++ b/ui/events/blink/blink_event_util.h
@@ -86,13 +86,13 @@ int WebEventModifiersToEventFlags(int modifiers);
 blink::WebInputEvent::Modifiers DomCodeToWebInputEventModifiers(
     ui::DomCode code);
 
-bool IsGestureScollOrPinch(blink::WebInputEvent::Type);
+bool IsGestureScrollFlingPinch(blink::WebInputEvent::Type);
 
 bool IsContinuousGestureEvent(blink::WebInputEvent::Type);
 
 inline const blink::WebGestureEvent& ToWebGestureEvent(
     const blink::WebInputEvent& event) {
-  DCHECK(IsGestureScollOrPinch(event.type()));
+  DCHECK(IsGestureScrollFlingPinch(event.type()));

[tdresser, 2017/03/10 15:31:06]

It isn't immediately obvious why this DCHECK is here. Should it have a comment?

[dtapuska, 2017/03/10 15:49:14]

I'd really expect the dcheck to be calling this:
https://cs.chromium.org/chromium/src/third_party/WebKit/public/platform/WebIn...

[chongz, 2017/03/14 14:51:06]

Done.

   return static_cast<const blink::WebGestureEvent&>(event);
 }