[Mac][MC] Allow right click to open the message center tray.

[Mac][MC] Allow right click to open the message center tray.

BUG=243937
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=202627

[Nico, 2013-05-28 16:05:07]

lgtm

https://codereview.chromium.org/16073003/diff/1/ui/message_center/cocoa/statu...
File ui/message_center/cocoa/status_item_view.mm (right):

https://codereview.chromium.org/16073003/diff/1/ui/message_center/cocoa/statu...
ui/message_center/cocoa/status_item_view.mm:78: }
otherMouseDown/Up: too?

[Robert Sesek, 2013-05-28 16:16:23]

https://codereview.chromium.org/16073003/diff/1/ui/message_center/cocoa/statu...
File ui/message_center/cocoa/status_item_view.mm (right):

https://codereview.chromium.org/16073003/diff/1/ui/message_center/cocoa/statu...
ui/message_center/cocoa/status_item_view.mm:78: }
On 2013/05/28 16:05:08, Nico wrote:
> otherMouseDown/Up: too?

Done.

[commit-bot: I haz the power, 2013-05-28 16:16:46]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/rsesek@chromium.org/16073003/6001

[commit-bot: I haz the power, 2013-05-28 16:19:19]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/rsesek@chromium.org/16073003/6001

[dewittj, 2013-05-28 16:23:07]

I think this code can trigger a DCHECK if the user clicks with left mouse, holds
it while clicking right mouse then releases both.

[Robert Sesek, 2013-05-28 16:28:46]

On 2013/05/28 16:23:07, dewittj wrote:
> I think this code can trigger a DCHECK if the user clicks with left mouse,
holds
> it while clicking right mouse then releases both.

You're right, though the likelihood of that happening is next to nil since Mac
users don't have multi-button mice. But I've removed the DCHECKs.

[dewittj, 2013-05-28 16:37:45]

lgtm

[commit-bot: I haz the power, 2013-05-28 16:38:12]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/rsesek@chromium.org/16073003/16002

[commit-bot: I haz the power, 2013-05-28 19:59:42]

Change committed as 202627