Revert of [VSync Queue] Plug touch ack to gesture events and flush vsync queue if necessary

Revert of [VSync Queue] Plug touch ack to gesture events and flush vsync queue if necessary (patchset #8 id:200001 of https://codereview.chromium.org/2869823003/ )

Reason for revert:
Speculative revert.

The builder
https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Win%20x64%20Bu...
fails with:

FAILED: gesture_detection.dll gesture_detection.dll.lib
gesture_detection.dll.pdb 
C:/b/depot_tools/python276_bin/python.exe
../../build/toolchain/win/tool_wrapper.py link-wrapper environment.x64 False
link.exe /nologo /IMPLIB:./gesture_detection.dll.lib /DLL
/OUT:./gesture_detection.dll /PDB:./gesture_detection.dll.pdb
@./gesture_detection.dll.rsp
LINK : fatal error LNK1104: cannot open file 'gfx_switches.dll.lib'

The reverted CL touches code with gestures and gfx.

Today's sheriff.

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-Original-Commit-Position: refs/heads/master@{#480202}
> Committed: https://chromium.googlesource.com/chromium/src/+/80553a96dc6712a24a7d0a3e3074d5b7f5f65338
> Review-Url: https://codereview.chromium.org/2869823003
> Cr-Commit-Position: refs/heads/master@{#481130}
> Committed: https://chromium.googlesource.com/chromium/src/+/44f78150b462c4c9daaba7ae47a772fba104e56a

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/2953563002
Cr-Commit-Position: refs/heads/master@{#481136}
Committed: https://chromium.googlesource.com/chromium/src/+/6b8c880ce554510082ac8aaff2d86e58dd6df4bc

[vabr (Chromium), 2017-06-21 07:43:31]

The CQ bit was checked by vabr@chromium.org

[vabr (Chromium), 2017-06-21 07:43:31]

Created Revert of [VSync Queue] Plug touch ack to gesture events and flush vsync
queue if necessary

[commit-bot: I haz the power, 2017-06-21 07:43:43]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-06-21 07:44:21]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1498031011423350, "parent_rev":
"d1e9baba16a19ca08fa1daa1f1cc6f929379fc44", "commit_rev":
"6b8c880ce554510082ac8aaff2d86e58dd6df4bc"}

[commit-bot: I haz the power, 2017-06-21 07:44:35]

Description was changed from

==========
Revert of [VSync Queue] Plug touch ack to gesture events and flush vsync queue
if necessary (patchset #8 id:200001 of
https://codereview.chromium.org/2869823003/ )

Reason for revert:
Speculative revert.

The builder
https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Win%20x64%20Bu...
fails with:

FAILED: gesture_detection.dll gesture_detection.dll.lib
gesture_detection.dll.pdb 
C:/b/depot_tools/python276_bin/python.exe
../../build/toolchain/win/tool_wrapper.py link-wrapper environment.x64 False
link.exe /nologo /IMPLIB:./gesture_detection.dll.lib /DLL
/OUT:./gesture_detection.dll /PDB:./gesture_detection.dll.pdb
@./gesture_detection.dll.rsp
LINK : fatal error LNK1104: cannot open file 'gfx_switches.dll.lib'

The reverted CL touches code with gestures and gfx.

Today's sheriff.

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-Original-Commit-Position: refs/heads/master@{#480202}
> Committed:
https://chromium.googlesource.com/chromium/src/+/80553a96dc6712a24a7d0a3e3074...
> Review-Url: https://codereview.chromium.org/2869823003
> Cr-Commit-Position: refs/heads/master@{#481130}
> Committed:
https://chromium.googlesource.com/chromium/src/+/44f78150b462c4c9daaba7ae47a7...

TBR=dtapuska@chromium.org,sadrul@chromium.org,pfeldman@chromium.org,thestig@c...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=704732
==========

to

==========
Revert of [VSync Queue] Plug touch ack to gesture events and flush vsync queue
if necessary (patchset #8 id:200001 of
https://codereview.chromium.org/2869823003/ )

Reason for revert:
Speculative revert.

The builder
https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Win%20x64%20Bu...
fails with:

FAILED: gesture_detection.dll gesture_detection.dll.lib
gesture_detection.dll.pdb 
C:/b/depot_tools/python276_bin/python.exe
../../build/toolchain/win/tool_wrapper.py link-wrapper environment.x64 False
link.exe /nologo /IMPLIB:./gesture_detection.dll.lib /DLL
/OUT:./gesture_detection.dll /PDB:./gesture_detection.dll.pdb
@./gesture_detection.dll.rsp
LINK : fatal error LNK1104: cannot open file 'gfx_switches.dll.lib'

The reverted CL touches code with gestures and gfx.

Today's sheriff.

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-Original-Commit-Position: refs/heads/master@{#480202}
> Committed:
https://chromium.googlesource.com/chromium/src/+/80553a96dc6712a24a7d0a3e3074...
> Review-Url: https://codereview.chromium.org/2869823003
> Cr-Commit-Position: refs/heads/master@{#481130}
> Committed:
https://chromium.googlesource.com/chromium/src/+/44f78150b462c4c9daaba7ae47a7...

TBR=dtapuska@chromium.org,sadrul@chromium.org,pfeldman@chromium.org,thestig@c...
# 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/2953563002
Cr-Commit-Position: refs/heads/master@{#481136}
Committed:
https://chromium.googlesource.com/chromium/src/+/6b8c880ce554510082ac8aaff2d8...
==========

[commit-bot: I haz the power, 2017-06-21 07:44:36]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/6b8c880ce554510082ac8aaff2d8...

[vabr (Chromium), 2017-06-21 08:05:53]

Strangely enough, the builder got green again before the revert made it into the
blamelist.

So I will reland this, but given the relation of the failing log to the touched
files, I'd like to ask the author or reviewers to consider whether the CL might
be causing a build flake.

Cheers,
Today's sheriff

[vabr (Chromium), 2017-06-21 08:06:26]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/2951973002/ by vabr@chromium.org.

The reason for reverting is: Builder recovered before the revert made it in..