Patch overlay panel event stream internally

Patch overlay panel event stream internally

This change adds logic inside the OverlayPanelEventFilter to make
sure that at least one down event is being passed to the internal
gesture detector before a stream starts.

BUG=715183
Review-Url: https://codereview.chromium.org/2843263003
Cr-Commit-Position: refs/heads/master@{#467768}
Committed: https://chromium.googlesource.com/chromium/src/+/5cf597d4e4e4409bf219d499ff00972fa60fcf0c

[mdjones, 2017-04-27 17:07:58]

mdjones@chromium.org changed reviewers:
+ khushalsagar@chromium.org

[mdjones, 2017-04-27 17:07:59]

ptal

[Khushal, 2017-04-27 17:18:00]

https://codereview.chromium.org/2843263003/diff/20001/chrome/android/java/src...
File
chrome/android/java/src/org/chromium/chrome/browser/compositor/layouts/eventfilter/OverlayPanelEventFilter.java
(right):

https://codereview.chromium.org/2843263003/diff/20001/chrome/android/java/src...
chrome/android/java/src/org/chromium/chrome/browser/compositor/layouts/eventfilter/OverlayPanelEventFilter.java:339:
super.onTouchEventInternal(down);
set mFilterHadDownEvent to true in this block as well.

[mdjones, 2017-04-27 17:22:32]

https://codereview.chromium.org/2843263003/diff/20001/chrome/android/java/src...
File
chrome/android/java/src/org/chromium/chrome/browser/compositor/layouts/eventfilter/OverlayPanelEventFilter.java
(right):

https://codereview.chromium.org/2843263003/diff/20001/chrome/android/java/src...
chrome/android/java/src/org/chromium/chrome/browser/compositor/layouts/eventfilter/OverlayPanelEventFilter.java:339:
super.onTouchEventInternal(down);
On 2017/04/27 17:18:00, Khushal wrote:
> set mFilterHadDownEvent to true in this block as well.

Whoops; done.

[Khushal, 2017-04-27 17:29:40]

lgtm and good luck!

[mdjones, 2017-04-27 18:46:00]

The CQ bit was checked by mdjones@chromium.org

[commit-bot: I haz the power, 2017-04-27 18:46:52]

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-04-27 20:29:53]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1493318760025880,
"parent_rev": "94ad18d772d3d2fbf8c084b93af70502432a4256", "commit_rev":
"5cf597d4e4e4409bf219d499ff00972fa60fcf0c"}

[commit-bot: I haz the power, 2017-04-27 20:31:18]

Description was changed from

==========
Patch overlay panel event stream internally

This change adds logic inside the OverlayPanelEventFilter to make
sure that at least one down event is being passed to the internal
gesture detector before a stream starts.

BUG=715183
==========

to

==========
Patch overlay panel event stream internally

This change adds logic inside the OverlayPanelEventFilter to make
sure that at least one down event is being passed to the internal
gesture detector before a stream starts.

BUG=715183

Review-Url: https://codereview.chromium.org/2843263003
Cr-Commit-Position: refs/heads/master@{#467768}
Committed:
https://chromium.googlesource.com/chromium/src/+/5cf597d4e4e4409bf219d499ff00...
==========

[commit-bot: I haz the power, 2017-04-27 20:31:19]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/5cf597d4e4e4409bf219d499ff00...