Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(643)

Issue 2770713006: Filter out stationary touches from coalesced list (Closed)

Created:
3 years, 9 months ago by Navid Zolghadr
Modified:
3 years, 9 months ago
Reviewers:
mustaq, dtapuska, bokan
CC:
chromium-reviews, blink-reviews, dtapuska+blinkwatch_chromium.org, Navid Zolghadr
Target Ref:
refs/heads/master
Project:
chromium
Visibility:
Public.

Description

Filter out stationary touches from coalesced list Since stationary touch points are added in our code and are not real event stream we should not add those to the list of coalesced events. This CL filters out all the stationary touch points from the coalesced event list of a touch pointer event. BUG=704419 Review-Url: https://codereview.chromium.org/2770713006 Cr-Commit-Position: refs/heads/master@{#459131} Committed: https://chromium.googlesource.com/chromium/src/+/910c2e27c11b75877fb552aa0162a7419944dfed

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -2 lines) Patch
M third_party/WebKit/Source/core/input/PointerEventManager.cpp View 1 chunk +2 lines, -2 lines 0 comments Download

Messages

Total messages: 17 (10 generated)
Navid Zolghadr
3 years, 9 months ago (2017-03-23 14:41:34 UTC) #4
mustaq
A test would be great but it seems hard through our current test infrastructure as ...
3 years, 9 months ago (2017-03-23 15:13:12 UTC) #5
Navid Zolghadr
3 years, 9 months ago (2017-03-23 15:26:01 UTC) #7
bokan
lgtm
3 years, 9 months ago (2017-03-23 15:30:59 UTC) #8
dtapuska
On 2017/03/23 15:26:01, Navid Zolghadr wrote: lgtm
3 years, 9 months ago (2017-03-23 15:31:06 UTC) #9
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2770713006/1
3 years, 9 months ago (2017-03-23 17:30:33 UTC) #13
commit-bot: I haz the power
3 years, 9 months ago (2017-03-23 17:43:17 UTC) #17
Message was sent while issue was closed.
Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/910c2e27c11b75877fb552aa0162...

Powered by Google App Engine
This is Rietveld 408576698