DescriptionRevert of [VSync Queue] Plug touch ack to gesture events and flush vsync queue if necessary (patchset #7 id:180001 of https://codereview.chromium.org/2869823003/ )
Reason for revert:
Likely causing failures on Linux ChromiumOS MSan Tests.
Original issue's description:
> [VSync Queue] Plug touch ack to gesture events and flush vsync queue if necessary
>
> For a gesture event, we want to know whether the source touch event was handled
> blocking or non-blocking, and we can flush the vsync queue immediately if it was
> blocking.
>
> This patch will help avoid regression in |first_gesture_scroll_update_latency|
> in |smoothness.tough_scrolling_cases|.
>
> After CL the process would be:
> 1. Browser -> Renderer: Send touch
> 2. Renderer -> Browser: Touch Ack
> 3. Browser: Set |is_source_touch_event_set_non_blocking == true| if the ack was
> |INPUT_EVENT_ACK_STATE_SET_NON_BLOCKING*|
> 4. Browser -> Renderer: Send generated gesture events with
> |is_source_touch_event_set_non_blocking| bit
> 5. Renderer: Flush vsync queue if |is_source_touch_event_set_non_blocking == true|.
>
> BUG=704732
>
> Review-Url: https://codereview.chromium.org/2869823003
> Cr-Commit-Position: refs/heads/master@{#480202}
> Committed: https://chromium.googlesource.com/chromium/src/+/80553a96dc6712a24a7d0a3e3074d5b7f5f65338
TBR=dtapuska@chromium.org,sadrul@chromium.org,pfeldman@chromium.org,thestig@chromium.org,chongz@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=704732
Review-Url: https://codereview.chromium.org/2943883002
Cr-Commit-Position: refs/heads/master@{#480281}
Committed: https://chromium.googlesource.com/chromium/src/+/07f0acbacf4853d4cfe56a134a601576fd547cf0
Patch Set 1 #Messages
Total messages: 6 (3 generated)
|