-
Notifications
You must be signed in to change notification settings - Fork 15k
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: prevent menu gc during popup #20745
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.
Can you rebase so that Windows CI runs properly? There was an issue where CI was getting skipped that has now been fixed.
4f171a9
to
8232e64
Compare
@jkleinsc should be set now! |
Release Notes Persisted
|
I was unable to backport this PR to "5-0-x" cleanly; |
I was unable to backport this PR to "7-0-x" cleanly; |
I was unable to backport this PR to "6-1-x" cleanly; |
I was unable to backport this PR to "8-x-y" cleanly; |
@codebytere has manually backported this PR to "8-x-y", please check out #20785 |
@codebytere has manually backported this PR to "6-1-x", please check out #20786 |
Description of Change
Fixes #20737.
Ensures that the garbage collector will track the object stored in the handle and they won't get gc'd accidentally while they're open.
cc @MarshallOfSound @zcbenz
Checklist
npm test
passesRelease Notes
Notes: Fixed a crash in Menus related to
menu.popup()
.