fix: respect the user's active app focus

[shazron]

Apple recommends not setting this to true, and this behaviour is non-standard (not to mention annoying when someone is actively focused on another app, like writing) https://developer.apple.com/documentation/appkit/nsapplication/1428468-activateignoringotherapps?language=objc

cc @zcbenz

Description of Change

Checklist

• PR description included and stakeholders cc'd
• npm test passes
  (https://github.com/electron/electron/blob/master/docs/development/testing.md)
• 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.

Release Notes

Notes: Focus methods no longer steal focus when users are active on other apps

[welcome]

💖 Thanks for opening this pull request! 💖

We use semantic commit messages to streamline the release process. Before your pull request can be merged, you should update your pull request title to start with a semantic prefix.

Examples of commit messages with semantic prefixes:

• fix: don't overwrite prevent_default if default wasn't prevented
• feat: add app.isPackaged() method
• docs: app.isDefaultProtocolClient is now available on Linux

Things that will help get your PR across the finish line:

• Follow the JavaScript, C++, and Python coding style.
• Run npm run lint locally to catch formatting errors earlier.
• Document any user-facing changes you've made following the documentation styleguide.
• Include tests when adding/changing behavior.
• Include screenshots and animated GIFs whenever possible.

We get a lot of pull requests on this repo, so please be patient and we will get back to you as soon as we can.

[MarshallOfSound]

One of these (the bundle mover) we definitely can't change.

The other two we probably can but I'd have to check the history of those impls. Tentative approval pending a check of the superview logic.

[MarshallOfSound]

This will break the bundle mover, can we not make this change here?

[shazron]

Reverted the change.

[codebytere]

@shazron can you please rebase this PR?

[shazron]

@codebytere Rebased.

[release-clerk]

Release Notes Persisted

Focus methods no longer steal focus when users are active on other apps

[welcome]

Congrats on merging your first pull request! 🎉🎉🎉