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 restoring minimized hidden window #21813
Conversation
c2bad81
to
d4823a9
Compare
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.
seems straightforward to me - lgtm
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.
Verified not to regress any previous behaviors.
@zcbenz what about 6-1-x
?
Release Notes Persisted
|
I have automatically backported this PR to "7-1-x", please check out #21820 |
I have automatically backported this PR to "8-x-y", please check out #21821 |
/trop run backport-to 6-1-x |
The backport process for this PR has been manually initiated - |
I was unable to backport this PR to "6-1-x" cleanly; |
@codebytere has manually backported this PR to "6-1-x", please check out #22153 |
Description of Change
Close #21614.
When invoking
SetWindowPlacement
inPreHandleMSG
it would trigger window message proc in Chromium, and then crash under some cases because Chromium does not expect its window message proc to be called in thePreHandleMSG
hook.Checklist
npm test
passesRelease Notes
Notes: Fix crash when restoring minimized hidden window on Windows.