Removed resetting of pinch-gesture filter state on pinch end.

Removed resetting of pinch-gesture filter state on pinch end.

We used to reset the TouchActionFilter state |drop_pinch_gesture_events_|
at a pinch end assuming that there is a single pinch-sequence within a
gesture-scroll sequence. This logically correct assumption turned out to
be wrong in practice: a pinch-zoom-out-then-zoom-in actually sends two
consecutive pinch-begin/end sequences between a gesture-scroll-begin/end
sequence.

This CL defers the resetting of |drop_pinch_gesture_events_| to
bracketing gesture-scroll-end to avoid wrong filtering during the second
pinch-begin/end sequence.

BUG=662047
Committed: https://crrev.com/b8dc459398704fe6df59e1be0655399085454449
Cr-Commit-Position: refs/heads/master@{#434248}

[mustaq, 2016-11-23 19:09:24]

Description was changed from

==========
WIP: Added LOGs for testing.

BUG=662047
==========

to

==========
Removed resetting of pinch-gesture filter state on pinch end.

We used to reset the TouchActionFilter state |drop_pinch_gesture_events_|
at a pinch end assuming that there is a single pinch-sequence within a
gesture-scroll sequence. This logically correct assumption turned out to
be wrong in practice: a pinch-zoom-out-then-zoom-in actually sends two
consecutive pinch-begin/end sequences between a gesture-scroll-begin/end
sequence.

This CL defers the resetting of |drop_pinch_gesture_events_| to 
bracketing gesture-scroll-end to avoid wrong filtering during the second
pinch-begin/end sequence.

BUG=662047
==========

[mustaq, 2016-11-23 19:09:44]

Patchset #2 (id:20001) has been deleted

[mustaq, 2016-11-23 19:32:28]

mustaq@chromium.org changed reviewers:
+ dtapuska@chromium.org

[mustaq, 2016-11-23 19:32:29]

ptal

[mustaq, 2016-11-23 19:36:28]

The CQ bit was checked by mustaq@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-11-23 19:37:11]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[dtapuska, 2016-11-23 19:53:47]

On 2016/11/23 19:32:29, mustaq wrote:
> ptal

lgtm

[mustaq, 2016-11-23 19:57:07]

The CQ bit was unchecked by mustaq@chromium.org

[mustaq, 2016-11-23 19:57:24]

The CQ bit was checked by mustaq@chromium.org

[commit-bot: I haz the power, 2016-11-23 19:57:54]

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, 2016-11-23 21:13:19]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1479931044534060,
"parent_rev": "4365c1bf915cac406537e6a0494891f8b9dd704e", "commit_rev":
"75cdaa8b41f8c52885dd74439b6f4ce2cd987e4b"}

[commit-bot: I haz the power, 2016-11-23 21:13:43]

Description was changed from

==========
Removed resetting of pinch-gesture filter state on pinch end.

We used to reset the TouchActionFilter state |drop_pinch_gesture_events_|
at a pinch end assuming that there is a single pinch-sequence within a
gesture-scroll sequence. This logically correct assumption turned out to
be wrong in practice: a pinch-zoom-out-then-zoom-in actually sends two
consecutive pinch-begin/end sequences between a gesture-scroll-begin/end
sequence.

This CL defers the resetting of |drop_pinch_gesture_events_| to 
bracketing gesture-scroll-end to avoid wrong filtering during the second
pinch-begin/end sequence.

BUG=662047
==========

to

==========
Removed resetting of pinch-gesture filter state on pinch end.

We used to reset the TouchActionFilter state |drop_pinch_gesture_events_|
at a pinch end assuming that there is a single pinch-sequence within a
gesture-scroll sequence. This logically correct assumption turned out to
be wrong in practice: a pinch-zoom-out-then-zoom-in actually sends two
consecutive pinch-begin/end sequences between a gesture-scroll-begin/end
sequence.

This CL defers the resetting of |drop_pinch_gesture_events_| to 
bracketing gesture-scroll-end to avoid wrong filtering during the second
pinch-begin/end sequence.

BUG=662047
==========

[commit-bot: I haz the power, 2016-11-23 21:13:44]

Committed patchset #3 (id:60001)

[commit-bot: I haz the power, 2016-11-23 21:16:24]

Description was changed from

==========
Removed resetting of pinch-gesture filter state on pinch end.

We used to reset the TouchActionFilter state |drop_pinch_gesture_events_|
at a pinch end assuming that there is a single pinch-sequence within a
gesture-scroll sequence. This logically correct assumption turned out to
be wrong in practice: a pinch-zoom-out-then-zoom-in actually sends two
consecutive pinch-begin/end sequences between a gesture-scroll-begin/end
sequence.

This CL defers the resetting of |drop_pinch_gesture_events_| to 
bracketing gesture-scroll-end to avoid wrong filtering during the second
pinch-begin/end sequence.

BUG=662047
==========

to

==========
Removed resetting of pinch-gesture filter state on pinch end.

We used to reset the TouchActionFilter state |drop_pinch_gesture_events_|
at a pinch end assuming that there is a single pinch-sequence within a
gesture-scroll sequence. This logically correct assumption turned out to
be wrong in practice: a pinch-zoom-out-then-zoom-in actually sends two
consecutive pinch-begin/end sequences between a gesture-scroll-begin/end
sequence.

This CL defers the resetting of |drop_pinch_gesture_events_| to
bracketing gesture-scroll-end to avoid wrong filtering during the second
pinch-begin/end sequence.

BUG=662047

Committed: https://crrev.com/b8dc459398704fe6df59e1be0655399085454449
Cr-Commit-Position: refs/heads/master@{#434248}
==========

[commit-bot: I haz the power, 2016-11-23 21:16:25]

Patchset 3 (id:??) landed as
https://crrev.com/b8dc459398704fe6df59e1be0655399085454449
Cr-Commit-Position: refs/heads/master@{#434248}