MD WebUI: Ensure the search field opens on touchscreen tap

MD WebUI: Ensure the search field opens on touchscreen tap

This fixes an issue where after touchscreen tapping the search icon, the
browser would return focus to the search icon, immediately closing the
search field. Using the native 'click' event instead of the simulated
'tap' event fixes this issue.

BUG=624356
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/93a84398c5b2804930aa71c542617d0beae1f470
Cr-Commit-Position: refs/heads/master@{#434905}

[tsergeant, 2016-11-28 00:54:36]

Description was changed from

==========
MD WebUI: Ensure the search field opens on touchscreen tap

This fixes an issue where after touchscreen tapping the search icon, the
browser would return focus to the search icon, immediately closing the
search field. Using the native 'click' event instead of the simulated
'tap' event fixes this issue.

BUG=624356
==========

to

==========
MD WebUI: Ensure the search field opens on touchscreen tap

This fixes an issue where after touchscreen tapping the search icon, the
browser would return focus to the search icon, immediately closing the
search field. Using the native 'click' event instead of the simulated
'tap' event fixes this issue.

BUG=624356
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[tsergeant, 2016-11-28 02:28:01]

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

[commit-bot: I haz the power, 2016-11-28 02:28:14]

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

[tsergeant, 2016-11-28 02:33:43]

tsergeant@chromium.org changed reviewers:
+ dbeam@chromium.org

[tsergeant, 2016-11-28 02:33:44]

PTAL!

[commit-bot: I haz the power, 2016-11-28 03:41:20]

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

[commit-bot: I haz the power, 2016-11-28 03:41:21]

Dry run: This issue passed the CQ dry run.

[tsergeant, 2016-11-28 23:28:35]

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

[commit-bot: I haz the power, 2016-11-28 23:29:43]

Dry run: 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-29 01:58:07]

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

[commit-bot: I haz the power, 2016-11-29 01:58:08]

Dry run: This issue passed the CQ dry run.

[Dan Beam, 2016-11-29 04:39:09]

lgtm

have you filed a bug or notified the Polymer team?  they've been using
sourceEvents lately to try to avoid these double-handling of tap and click bugs
(that I think this is hitting)

[Dan Beam, 2016-11-29 04:40:56]

On 2016/11/29 04:39:09, Dan Beam wrote:
> lgtm
> 
> have you filed a bug or notified the Polymer team?  they've been using
> sourceEvents lately to try to avoid these double-handling of tap and click
bugs
> (that I think this is hitting)

s/sourceEvents/sourceCapabilities/
https://developer.mozilla.org/en-US/docs/Web/API/UIEvent/sourceCapabilities

[tsergeant, 2016-11-29 05:42:43]

The CQ bit was checked by tsergeant@chromium.org

[commit-bot: I haz the power, 2016-11-29 05:42:58]

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

[tsergeant, 2016-11-29 05:44:47]

On 2016/11/29 04:39:09, Dan Beam wrote:
> lgtm
> 
> have you filed a bug or notified the Polymer team?  they've been using
> sourceEvents lately to try to avoid these double-handling of tap and click
bugs
> (that I think this is hitting)

I haven't yet, I'll follow up with a Polymer bug.

[commit-bot: I haz the power, 2016-11-29 05:47:13]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1480398163212770,
"parent_rev": "971ef57a5f816d985e395554893ec150217a00f9", "commit_rev":
"8bff0ae7572cc3c2c0dc1c41a08105ba5fda52f8"}

[commit-bot: I haz the power, 2016-11-29 05:47:42]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-11-29 05:49:18]

Description was changed from

==========
MD WebUI: Ensure the search field opens on touchscreen tap

This fixes an issue where after touchscreen tapping the search icon, the
browser would return focus to the search icon, immediately closing the
search field. Using the native 'click' event instead of the simulated
'tap' event fixes this issue.

BUG=624356
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
MD WebUI: Ensure the search field opens on touchscreen tap

This fixes an issue where after touchscreen tapping the search icon, the
browser would return focus to the search icon, immediately closing the
search field. Using the native 'click' event instead of the simulated
'tap' event fixes this issue.

BUG=624356
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/93a84398c5b2804930aa71c542617d0beae1f470
Cr-Commit-Position: refs/heads/master@{#434905}
==========

[commit-bot: I haz the power, 2016-11-29 05:49:19]

Patchset 2 (id:??) landed as
https://crrev.com/93a84398c5b2804930aa71c542617d0beae1f470
Cr-Commit-Position: refs/heads/master@{#434905}