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

ui/events/blink/blink_event_util.h

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef UI_EVENTS_BLINK_BLINK_EVENT_UTIL_H_
 #define UI_EVENTS_BLINK_BLINK_EVENT_UTIL_H_
 
 #include <memory>
 
 #include "third_party/WebKit/public/platform/WebGestureEvent.h"
@@ skipping 68 matching lines @@
     float orientation_rad,
     float tilt_rad,
     int android_buttons_changed,
     int tool_type);
 
 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);
 }
 
 }  // namespace ui
 
 #endif  // UI_EVENTS_BLINK_BLINK_EVENT_UTIL_H_