fix: double traffic lights on exit fullscreen

[codebytere]

Description of Change

Closes #30026.

Fixes an issue where the traffic lights would get double-drawn when exiting fullscreen and adding a BrowserView on macOS. In 8bf66f8 the original code did not redraw traffic lights when the window was fullscreened, but this logic did not perisist and then when I merged #29595 this created the issue.

We fix this by not reordering the buttons in the view hierarchy when we're in fullscreen mode.

Tested with https://gist.github.com/45f0a58d69bec2834535e57768363572 and confirmed not to regress #29541.

Checklist

• PR description included and stakeholders cc'd
• npm test passes
• PR release notes describe the change in a way relevant to app developers, and are capitalized, punctuated, and past tense.

Release Notes

Notes: Fixes an issue where the traffic lights would get double-drawn when exiting fullscreen and adding a BrowserView on macOS.

[ckerr]

Gotta love one-liners

[codebytere]

Failure not relevant.

[release-clerk]

Release Notes Persisted

Fixes an issue where the traffic lights would get double-drawn when exiting fullscreen and adding a BrowserView on macOS.

[trop]

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

[trop]

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