[Compositor event queue] Coalesce gesture scroll&pinch of the same sequence

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"
 #include "third_party/WebKit/public/platform/WebInputEvent.h"
 #include "ui/events/gesture_detection/motion_event.h"
 
 namespace blink {
-class WebGestureEvent;
-class WebInputEvent;
 class WebTouchEvent;
 }
 
 namespace gfx {
 class PointF;
 class Vector2d;
 }
 
 namespace ui {
 enum class DomCode;
 struct GestureEventData;
 struct GestureEventDetails;
 class MotionEvent;
 
 bool CanCoalesce(const blink::WebInputEvent& event_to_coalesce,
                  const blink::WebInputEvent& event);
 
 void Coalesce(const blink::WebInputEvent& event_to_coalesce,
               blink::WebInputEvent* event);
 
+bool IsCompatibleScrollorPinch(const blink::WebGestureEvent& new_event,
+                               const blink::WebGestureEvent& event_in_queue);
+
+// Coalesces 3 GestureScroll/PinchUpdate into 2 events.
+// Returns <GestureScrollUpdate, GesturePinchUpdate>.
+std::pair<blink::WebGestureEvent, blink::WebGestureEvent>
+CoalesceScrollAndPinch(const blink::WebGestureEvent* second_last_event,
+                       const blink::WebGestureEvent& last_event,
+                       const blink::WebGestureEvent& new_event);
+
 blink::WebTouchEvent CreateWebTouchEventFromMotionEvent(
     const MotionEvent& event,
     bool may_cause_scrolling);
 
 blink::WebGestureEvent CreateWebGestureEvent(const GestureEventDetails& details,
                                              base::TimeTicks timestamp,
                                              const gfx::PointF& location,
                                              const gfx::PointF& raw_location,
                                              int flags,
                                              uint32_t unique_touch_event_id);
@@ skipping 31 matching lines @@
 
 int WebEventModifiersToEventFlags(int modifiers);
 
 blink::WebInputEvent::Modifiers DomCodeToWebInputEventModifiers(
     ui::DomCode code);
 
 bool IsGestureScollOrPinch(blink::WebInputEvent::Type);
 
 bool IsContinuousGestureEvent(blink::WebInputEvent::Type);
 
+inline const blink::WebGestureEvent* ToWebGestureEvent(

[tdresser, 2016/12/13 17:18:19]

The asymmetry of these methods is a bit tricky. Could we make it so neither or
both of them take a pointer?

[chongz, 2016/12/13 18:55:45]

Removed the pointer version. 
However I'm not entirely sure if that's what you mean because both methods are
converting |WebInputEvent| to |WebGestureEvent|.

+    const blink::WebInputEvent* event) {
+  DCHECK(!event || IsGestureScollOrPinch(event->type));
+  return static_cast<const blink::WebGestureEvent*>(event);
+}
+
+inline const blink::WebGestureEvent& ToWebGestureEvent(
+    const blink::WebInputEvent& event) {
+  DCHECK(IsGestureScollOrPinch(event.type));
+  return static_cast<const blink::WebGestureEvent&>(event);
+}
+
 }  // namespace ui
 
 #endif  // UI_EVENTS_BLINK_BLINK_EVENT_UTIL_H_