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

ui/events/blink/blink_event_util.cc

 // 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.
 
 // MSVC++ requires this to be set before any other includes to get M_PI.
 #define _USE_MATH_DEFINES
 
 #include "ui/events/blink/blink_event_util.h"
 
 #include <stddef.h>
@@ skipping 944 matching lines @@
     case DomKeyLocation::RIGHT:
       return blink::WebInputEvent::IsRight;
     case DomKeyLocation::NUMPAD:
       return blink::WebInputEvent::IsKeyPad;
     case DomKeyLocation::STANDARD:
       break;
   }
   return static_cast<blink::WebInputEvent::Modifiers>(0);
 }
 
-bool IsGestureScollOrPinch(WebInputEvent::Type type) {
+bool IsGestureScrollFlingOrPinch(WebInputEvent::Type type) {

[dtapuska, 2017/03/14 15:06:51]

Isn't it IsGestureScrollOrFlingOrPinch?

[tdresser, 2017/03/14 15:27:15]

I was reading it with an implicit "," between Scroll and Fling.
Either way.

[chongz, 2017/03/14 17:51:24]

I've renamed it to |IsGestureScrollOrFlingOrPinch()| so readers don't need to
look for implementation or comments.

   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:
       return true;
     default:
       return false;
   }
 }
 
 bool IsContinuousGestureEvent(WebInputEvent::Type type) {
   switch (type) {
     case blink::WebGestureEvent::GestureScrollUpdate:
     case blink::WebGestureEvent::GesturePinchUpdate:
       return true;
     default:
       return false;
   }
 }
 
 }  // namespace ui