Update MenuButton closing time for async

Update MenuButton closing time for async

MenuButton rejects openning new menus when a reposted event arrives too
close to the closing time of the menu. However this calculation was based
on menus always being synchronous.

Updating MenuButton to set the closed time based on the pressed state being
decremented to 0. This works for both synchronous and async menus.

TEST=manual testing of translate bubble, and other menu buttons, views_unittests
BUG=642254
Committed: https://crrev.com/df1837b274e20b9ce31a03acd625bdd1775d71c7
Cr-Commit-Position: refs/heads/master@{#415636}

[jonross, 2016-08-30 22:26:41]

jonross@chromium.org changed reviewers:
+ sky@chromium.org

[jonross, 2016-08-30 22:26:42]

Hey,

As you thought offline, MenuButton's pre-existing decrement method handles the
issue perfectly fine.

Could you take a look?

Thanks,
Jon

[sky, 2016-08-30 22:40:12]

Nice. I like easy. LGTM

[jonross, 2016-08-31 13:18:05]

The CQ bit was checked by jonross@chromium.org

[commit-bot: I haz the power, 2016-08-31 13:18:21]

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-08-31 15:18:08]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-31 15:19:41]

Description was changed from

==========
Update MenuButton closing time for async

MenuButton rejects openning new menus when a reposted event arrives too
close to the closing time of the menu. However this calculation was based
on menus always being synchronous.

Updating MenuButton to set the closed time based on the pressed state being
decremented to 0. This works for both synchronous and async menus.

TEST=manual testing of translate bubble, and other menu buttons, views_unittests
BUG=642254
==========

to

==========
Update MenuButton closing time for async

MenuButton rejects openning new menus when a reposted event arrives too
close to the closing time of the menu. However this calculation was based
on menus always being synchronous.

Updating MenuButton to set the closed time based on the pressed state being
decremented to 0. This works for both synchronous and async menus.

TEST=manual testing of translate bubble, and other menu buttons, views_unittests
BUG=642254

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

[commit-bot: I haz the power, 2016-08-31 15:19:41]

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