Touch scrolling in list picker shouldn't close the popup

Touch scrolling in list picker shouldn't close the popup

Touch dragging should select the item under the finger only if the popup is not scrolling. This matches IE behavior.

When touch begins, we enter "touch select mode". And when a scroll event is fired we disengage "touch select mode".

BUG=493033
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=200894

[keishi, 2015-08-14 05:50:41]

keishi@chromium.org changed reviewers:
+ tkent@chromium.org

[keishi, 2015-08-14 05:50:41]

PTAL

[tkent, 2015-08-14 05:55:44]

lgtm

[tkent, 2015-08-14 05:55:57]

Thank you for fixing it!

[tkent, 2015-08-14 09:04:46]

The CQ bit was checked by tkent@chromium.org

[commit-bot: I haz the power, 2015-08-14 09:04:52]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1287003002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1287003002/20001

[commit-bot: I haz the power, 2015-08-14 09:58:58]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-08-14 09:58:58]

Try jobs failed on following builders:
  mac_chromium_rel_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[keishi, 2015-08-20 04:53:56]

I was seeing selection moving while scrolling on Ubuntu. I confirmed that this
is a general problem on Ubuntu where mousemove events fires sporadically while
touchmove. This is not a problem on Windows where no mousemove events fire while
touchmove.
Since there is no easy fix and the problem is outside the ListPicker and may be
specific to the touchscreen driver for my laptop, I am going to land this CL as
is.

[keishi, 2015-08-20 04:54:02]

The CQ bit was checked by keishi@chromium.org

[keishi, 2015-08-20 04:54:03]

The patchset sent to the CQ was uploaded after l-g-t-m from tkent@chromium.org
Link to the patchset: https://codereview.chromium.org/1287003002/#ps40001
(title: "rebaseline mac test")

[commit-bot: I haz the power, 2015-08-20 04:54:10]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1287003002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1287003002/40001

[commit-bot: I haz the power, 2015-08-20 05:25:46]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-08-20 05:25:46]

Try jobs failed on following builders:
  chromeos_daisy_chromium_compile_only_ng on tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_daisy_...)

[keishi, 2015-08-20 05:32:05]

The CQ bit was checked by keishi@chromium.org

[commit-bot: I haz the power, 2015-08-20 05:32:17]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1287003002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1287003002/40001

[commit-bot: I haz the power, 2015-08-20 05:58:39]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-08-20 05:58:40]

Try jobs failed on following builders:
  chromeos_daisy_chromium_compile_only_ng on tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_daisy_...)

[keishi, 2015-08-20 09:27:01]

The CQ bit was checked by keishi@chromium.org

[commit-bot: I haz the power, 2015-08-20 09:27:17]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1287003002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1287003002/40001

[commit-bot: I haz the power, 2015-08-20 10:26:27]

Committed patchset #3 (id:40001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=200894