fix: emit click events with tray context menu

[codebytere]

Description of Change

Closes #24196.

When a context menu is added to a Tray on macOS, the menu controller overrides the mouseUp handler, where click events were emitted. This fixes the issue by pulling click event handling into a separate function, and emitting click events on mouseDown when a menu controller has been set and normally on mouseUp otherwise.

Tested with https://gist.github.com/1ffed70969360d362c807d60fe692404.

cc @MarshallOfSound @nornagon @zcbenz

Checklist

• PR description included and stakeholders cc'd
• npm test passes
• PR title follows semantic commit guidelines
• PR release notes describe the change in a way relevant to app developers, and are capitalized, punctuated, and past tense.
• This is NOT A BREAKING CHANGE. Breaking changes may not be merged to master until 11-x-y is branched.

Release Notes

Notes: Fixed an issue with click events not being emitted on macOS for Trays with context menus set.

[release-clerk]

Release Notes Persisted

Fixed an issue with click events not being emitted on macOS for Trays with context menus set.

[trop]

I was unable to backport this PR to "8-x-y" cleanly;
you will need to perform this backport manually.

[trop]

I have automatically backported this PR to "10-x-y", please check out #24235

[trop]

I have automatically backported this PR to "9-x-y", please check out #24236

[trop]

@codebytere has manually backported this PR to "8-x-y", please check out #24239