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: offscreen rendering crash on input select #34069
Conversation
e8d26c6
to
5d31d0f
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 legit
Release Notes Persisted
|
I have automatically backported this PR to "17-x-y", please check out #34091 |
I have automatically backported this PR to "18-x-y", please check out #34092 |
I have automatically backported this PR to "19-x-y", please check out #34093 |
Description of Change
Closes #34047.
Fixes the following crash:
This was happening because we were calling
SizeChanged
on the video consumer for popup widgets, which would have invalidgfx::Size
values for their positions. To fix this, I refactored this to more closely mimic CEF's approach and callpaths, found here.Tested with https://gist.github.com/gtalusan/c0ad56685aeb3b2c1108fbccf6b451b4.
Checklist
npm test
passesRelease Notes
Notes: Fixed an OSR crash happening when input select items were rendered.