New issue
Advanced search Search tips

Issue 796630 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Dec 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Gesture ShowPress/Cancel unterminated in trace from Android device

Project Member Reported by dtapu...@chromium.org, Dec 20 2017

Issue description

Capture a input latency trace from Android ToT.

See that that GestureShowPress/Cancel are unterminated in InputLatency async events.


 
I believe this is a problem with the touch action filter and fling controller supressing the events but not calling the client API that they are discarding an event.
Project Member

Comment 2 by bugdroid1@chromium.org, Dec 20 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/e0c27aa594d4584e9dff30a0b6e3604089ad9a0c

commit e0c27aa594d4584e9dff30a0b6e3604089ad9a0c
Author: Dave Tapuska <dtapuska@chromium.org>
Date: Wed Dec 20 22:50:56 2017

Fix input latency async trace events that don't end.

TouchAction and Touch Screen suppression failed to ack the event
back to the client which caused latency info objects not to be
terminated.

BUG= 796630 

Change-Id: I0768d0cc83b346612ecc72a3f8e75d46874194cf
Reviewed-on: https://chromium-review.googlesource.com/837470
Reviewed-by: Timothy Dresser <tdresser@chromium.org>
Commit-Queue: Dave Tapuska <dtapuska@chromium.org>
Cr-Commit-Position: refs/heads/master@{#525501}
[modify] https://crrev.com/e0c27aa594d4584e9dff30a0b6e3604089ad9a0c/content/browser/renderer_host/input/gesture_event_queue.cc
[modify] https://crrev.com/e0c27aa594d4584e9dff30a0b6e3604089ad9a0c/content/browser/renderer_host/input/gesture_event_queue.h
[modify] https://crrev.com/e0c27aa594d4584e9dff30a0b6e3604089ad9a0c/content/browser/renderer_host/input/input_router_impl.cc

Status: Fixed (was: Assigned)

Sign in to add a comment