• Marijn Kruisselbrink's avatar
    Revert "synthetic gesture: Allow high-frequency dispatch." · ae26a2e4
    Marijn Kruisselbrink authored
    This reverts commit 581a728b.
    
    Reason for revert: Causing significant test flakiness in telemetry.internal.actions.scroll_unittest.ScrollActionTest.testScrollDistanceSlowWheel.
    
    Original change's description:
    > synthetic gesture: Allow high-frequency dispatch.
    > 
    > For synthesized scroll/drag/pinch gestures, dispatch the events
    > at a high frequency (120Hz) so that there's at least one event in
    > each frame. Dispatching events at a lower frequency (60Hz) means
    > the timer can go out of sync with begin-frame, and so there can
    > be frames where there was to event dispatch. Our telemetry code
    > reports this as jank, which is incorrect. Dispatching at a higher
    > frequency for the scroll/drag/pinch gestures resolves this issue.
    > 
    > BUG=783034
    > 
    > Change-Id: I1102f76ed743231cbb10fdb872e8c66f895fa223
    > Reviewed-on: https://chromium-review.googlesource.com/c/1255152
    > Reviewed-by: Victor Miura <vmiura@chromium.org>
    > Reviewed-by: Navid Zolghadr <nzolghadr@chromium.org>
    > Reviewed-by: Lan Wei <lanwei@chromium.org>
    > Commit-Queue: Sadrul Chowdhury <sadrul@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#596507}
    
    TBR=sadrul@chromium.org,vmiura@chromium.org,lanwei@chromium.org,nzolghadr@chromium.org
    
    Change-Id: I1e4d64dfc3b98a547c38f8f82a2dcd45141fd66b
    No-Presubmit: true
    No-Tree-Checks: true
    No-Try: true
    Bug: 783034, 892036
    Reviewed-on: https://chromium-review.googlesource.com/c/1261972Reviewed-by: default avatarMarijn Kruisselbrink <mek@chromium.org>
    Commit-Queue: Marijn Kruisselbrink <mek@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#596748}
    ae26a2e4
synthetic_tap_gesture.h 1.61 KB