MD History: Do not collapse synced device card when tapping menu button

MD History: Use click instead of tap events for synced device collapsing

This fixes an issue where using touch to open the synced device actions
menu would also collapse the device card, since one event used on-tap
and the other used on-click.

BUG=652634, 650551
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/95f84bba89a39f5d70372bb08dfb159208f6b6c0
Cr-Commit-Position: refs/heads/master@{#423091}

[tsergeant, 2016-10-05 00:59:30]

Description was changed from

==========
MD History: Do not collapse synced device card when tapping menu button

BUG=652634
==========

to

==========
MD History: Do not collapse synced device card when tapping menu button

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

[tsergeant, 2016-10-05 00:59:45]

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

[commit-bot: I haz the power, 2016-10-05 01:00:04]

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-10-05 02:03:39]

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

[commit-bot: I haz the power, 2016-10-05 02:03:40]

Dry run: This issue passed the CQ dry run.

[tsergeant, 2016-10-05 02:04:55]

Description was changed from

==========
MD History: Do not collapse synced device card when tapping menu button

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

to

==========
MD History: Use click instead of tap events for synced device collapsing

This fixes an issue where using touch to open the synced device actions
menu would also collapse the device card, since one event used on-tap
and the other used on-click.

BUG=652634,650551
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[tsergeant, 2016-10-05 02:06:24]

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

[tsergeant, 2016-10-05 02:06:24]

https://codereview.chromium.org/2389393002/diff/1/chrome/test/data/webui/md_h...
File chrome/test/data/webui/md_history/history_synced_tabs_test.js (right):

https://codereview.chromium.org/2389393002/diff/1/chrome/test/data/webui/md_h...
chrome/test/data/webui/md_history/history_synced_tabs_test.js:254: test('show
actions menu', function() {
Sadly, I have not been able to get MockInteractions to trigger the right events
to do an actual regression test of this.

[calamity, 2016-10-05 04:45:53]

lgtm

https://codereview.chromium.org/2389393002/diff/1/chrome/test/data/webui/md_h...
File chrome/test/data/webui/md_history/history_synced_tabs_test.js (right):

https://codereview.chromium.org/2389393002/diff/1/chrome/test/data/webui/md_h...
chrome/test/data/webui/md_history/history_synced_tabs_test.js:254: test('show
actions menu', function() {
On 2016/10/05 02:06:24, tsergeant wrote:
> Sadly, I have not been able to get MockInteractions to trigger the right
events
> to do an actual regression test of this.

=(

[tsergeant, 2016-10-05 06:25:21]

The CQ bit was checked by tsergeant@chromium.org

[commit-bot: I haz the power, 2016-10-05 06:25:40]

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-10-05 06:31:07]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-10-05 06:32:53]

Description was changed from

==========
MD History: Use click instead of tap events for synced device collapsing

This fixes an issue where using touch to open the synced device actions
menu would also collapse the device card, since one event used on-tap
and the other used on-click.

BUG=652634,650551
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
MD History: Use click instead of tap events for synced device collapsing

This fixes an issue where using touch to open the synced device actions
menu would also collapse the device card, since one event used on-tap
and the other used on-click.

BUG=652634,650551
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/95f84bba89a39f5d70372bb08dfb159208f6b6c0
Cr-Commit-Position: refs/heads/master@{#423091}
==========

[commit-bot: I haz the power, 2016-10-05 06:32:54]

Patchset 1 (id:??) landed as
https://crrev.com/95f84bba89a39f5d70372bb08dfb159208f6b6c0
Cr-Commit-Position: refs/heads/master@{#423091}