Touch Events - changedTouches list includes non-changed touch points on Android

content/browser/renderer_host/input/touch_event_queue.h

 // Copyright 2013 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 CONTENT_BROWSER_RENDERER_HOST_INPUT_TOUCH_EVENT_QUEUE_H_
 #define CONTENT_BROWSER_RENDERER_HOST_INPUT_TOUCH_EVENT_QUEUE_H_
 
 #include <deque>
 #include <map>
 
@@ skipping 150 matching lines @@
 
   // Ack all coalesced events in |acked_event| to the client with |ack_result|,
   // updating the acked events with |optional_latency_info| if it exists.
   void AckTouchEventToClient(InputEventAckState ack_result,
                              scoped_ptr<CoalescedWebTouchEvent> acked_event,
                              const ui::LatencyInfo* optional_latency_info);
 
   // Safely pop the head of the queue.
   scoped_ptr<CoalescedWebTouchEvent> PopTouchEvent();
 
-  // Dispatch |touch| to the client.
-  void SendTouchEventImmediately(const TouchEventWithLatencyInfo& touch);
+  // Dispatch |touch| to the client. Before dispatching, updates touches state

[jdduke (slow), 2015/01/14 16:04:33]

Perhaps "...updates pointer states in touchmove events for pointers that have
not changed position.", ending with period.

[USE s.singapati at gmail.com, 2015/01/16 14:14:28]

Done.

+  // in touchmove event if touch position has not been changed
+  void SendTouchEventImmediately(TouchEventWithLatencyInfo* touch);
 
   enum PreFilterResult {
     ACK_WITH_NO_CONSUMER_EXISTS,
     ACK_WITH_NOT_CONSUMED,
     FORWARD_TO_RENDERER,
   };
   // Filter touches prior to forwarding to the renderer, e.g., if the renderer
   // has no touch handler.
   PreFilterResult FilterBeforeForwarding(const blink::WebTouchEvent& event);
   void ForwardToRenderer(const TouchEventWithLatencyInfo& event);
@@ skipping 51 matching lines @@
   bool send_touch_events_async_;
   bool needs_async_touchmove_for_outer_slop_region_;
   scoped_ptr<TouchEventWithLatencyInfo> pending_async_touchmove_;
   double last_sent_touch_timestamp_sec_;
 
   // How touch events are handled during scrolling.  For now this is a global
   // setting for experimentation, but we may evolve it into an app-controlled
   // mode.
   const TouchScrollingMode touch_scrolling_mode_;
 
+  // Touch event is saved to compare touches position in next TouchMove event

[jdduke (slow), 2015/01/14 16:04:33]

Maybe "... to compare pointer positions for new touchmove events.", again ending
with period.

[USE s.singapati at gmail.com, 2015/01/16 14:14:28]

Done.

+  scoped_ptr<blink::WebTouchEvent> last_sent_touchevent_;
+
   DISALLOW_COPY_AND_ASSIGN(TouchEventQueue);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_RENDERER_HOST_INPUT_TOUCH_EVENT_QUEUE_H_