Ignore mouseup shortly after showing cr.ui.Menu

Ignore mouseup shortly after showing cr.ui.Menu

This works simlarly to views::MenuController[1] by ignoring mouseups:
- less than 200ms after showing some UI
- with minimal mouse movement (4px length difference)

R=mustaq@chromium.org
BUG=531939
[1] https://goo.gl/iZht5k

Committed: https://crrev.com/70aa692d68ee86d365928edd160c3575fda2b453
Cr-Commit-Position: refs/heads/master@{#350323}

[Dan Beam, 2015-09-22 03:39:28]

Patchset #1 (id:1) has been deleted

[Dan Beam, 2015-09-22 03:43:24]

Patchset #2 (id:40001) has been deleted

[Dan Beam, 2015-09-22 03:43:28]

Patchset #1 (id:20001) has been deleted

[mustaq, 2015-09-22 14:20:37]

Thanks Dan, this is clearly a better solution than mine.

LGTM with nits.

https://codereview.chromium.org/1358893003/diff/60001/ui/webui/resources/js/c...
File ui/webui/resources/js/cr/ui/menu.js (right):

https://codereview.chromium.org/1358893003/diff/60001/ui/webui/resources/js/c...
ui/webui/resources/js/cr/ui/menu.js:118: * @param {Event} e
Please add a brief doc.

https://codereview.chromium.org/1358893003/diff/60001/ui/webui/resources/js/c...
ui/webui/resources/js/cr/ui/menu.js:132: if (Math.sqrt(Math.pow(xDelta, 2) +
Math.pow(yDelta, 2)) > 4)
What about a faster+simpler math? This is a heuristic anyway. E.g.
  Math.abs(xDelta) + Math.abs(yDelta) > 4

[Dan Beam, 2015-09-22 15:51:31]

https://codereview.chromium.org/1358893003/diff/60001/ui/webui/resources/js/c...
File ui/webui/resources/js/cr/ui/menu.js (right):

https://codereview.chromium.org/1358893003/diff/60001/ui/webui/resources/js/c...
ui/webui/resources/js/cr/ui/menu.js:118: * @param {Event} e
On 2015/09/22 14:20:36, mustaq wrote:
> Please add a brief doc.

Done.

https://codereview.chromium.org/1358893003/diff/60001/ui/webui/resources/js/c...
ui/webui/resources/js/cr/ui/menu.js:132: if (Math.sqrt(Math.pow(xDelta, 2) +
Math.pow(yDelta, 2)) > 4)
On 2015/09/22 14:20:36, mustaq wrote:
> What about a faster+simpler math? This is a heuristic anyway. E.g.
>   Math.abs(xDelta) + Math.abs(yDelta) > 4

Done.

[Dan Beam, 2015-09-22 15:52:39]

Patchset #2 (id:80001) has been deleted

[Dan Beam, 2015-09-22 15:54:15]

The CQ bit was checked by dbeam@chromium.org

[Dan Beam, 2015-09-22 15:54:15]

The patchset sent to the CQ was uploaded after l-g-t-m from mustaq@chromium.org
Link to the patchset: https://codereview.chromium.org/1358893003/#ps100001
(title: "nit")

[commit-bot: I haz the power, 2015-09-22 15:55:07]

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

[commit-bot: I haz the power, 2015-09-22 17:02:02]

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

[commit-bot: I haz the power, 2015-09-22 17:02:03]

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

[Dan Beam, 2015-09-22 17:04:50]

The CQ bit was checked by dbeam@chromium.org

[commit-bot: I haz the power, 2015-09-22 17:05:38]

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

[commit-bot: I haz the power, 2015-09-22 18:22:10]

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

[commit-bot: I haz the power, 2015-09-22 18:22:10]

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

[Dan Beam, 2015-09-23 05:46:27]

Patchset #3 (id:120001) has been deleted

[Dan Beam, 2015-09-23 05:49:26]

Patchset #3 (id:140001) has been deleted

[Dan Beam, 2015-09-23 05:49:43]

The CQ bit was checked by dbeam@chromium.org

[Dan Beam, 2015-09-23 05:49:43]

The patchset sent to the CQ was uploaded after l-g-t-m from mustaq@chromium.org
Link to the patchset: https://codereview.chromium.org/1358893003/#ps160001
(title: "fix context menus")

[commit-bot: I haz the power, 2015-09-23 05:49:54]

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

[commit-bot: I haz the power, 2015-09-23 07:04:55]

Committed patchset #3 (id:160001)

[commit-bot: I haz the power, 2015-09-23 07:05:31]

Patchset 3 (id:??) landed as
https://crrev.com/70aa692d68ee86d365928edd160c3575fda2b453
Cr-Commit-Position: refs/heads/master@{#350323}