New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: crash when WindowButtonsProxy references cleared NSWindow #33069
fix: crash when WindowButtonsProxy references cleared NSWindow #33069
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@gerhardberger can you rebase your PR on the latest from main? I believe that will resolve the failing CI/tests.
2b23684
to
38e45a2
Compare
38e45a2
to
572f259
Compare
Release Notes Persisted
|
I have automatically backported this PR to "15-x-y", please check out #33487 |
I have automatically backported this PR to "17-x-y", please check out #33488 |
I have automatically backported this PR to "16-x-y", please check out #33489 |
I have automatically backported this PR to "18-x-y", please check out #33490 |
…ron#33069) * resets WindowButtonsProxy on window delete on macOS * fixes reset
…ron#33069) * resets WindowButtonsProxy on window delete on macOS * fixes reset
Description of Change
This PR resets the
WindowButtonsProxy
inNativeWindowMac
when deleting the underlyingNSWindow
. This is needed, becauseWindowButtonsProxy
has a reference toNSWindow
, which is not cleared whenNSWindow
is. This can cause a crash whenWindowButtonsProxy
calls something on theNSWindow
when that is already cleared.We ran into this crash at Around only when we were updating our electron app and wanted to quit the application and restart a new one. So it's hard to formulate a repro for this.
Checklist
Release Notes
Notes: Fix crash when WindowButtonsProxy references cleared NSWindow