Skip to content
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 on missing RenderWidgetHostView #38100

Merged
merged 1 commit into from Apr 25, 2023
Merged

Conversation

codebytere
Copy link
Member

@codebytere codebytere commented Apr 24, 2023

Description of Change

Discovered while investigating #37453.

Fixes a potential crash when right-clicking on macOS windows with draggable regions. This can happen if for example the main window never loads content - we need to guard for missing RenderWidgetHostView.

Checklist

Release Notes

Notes: Fixed a potential crash when right-clicking on macOS windows with draggable regions.

@electron-cation electron-cation bot added the new-pr 🌱 PR opened in the last 24 hours label Apr 24, 2023
@codebytere codebytere changed the title chore: fix crash on missing RenderWidgetHostView chore: fix crash on missing RenderWidgetHostView Apr 24, 2023
@codebytere codebytere added semver/patch backwards-compatible bug fixes target/25-x-y PR should also be added to the "25-x-y" branch. labels Apr 24, 2023
@codebytere codebytere changed the title chore: fix crash on missing RenderWidgetHostView fix: crash on missing RenderWidgetHostView Apr 24, 2023
@electron-cation electron-cation bot removed the new-pr 🌱 PR opened in the last 24 hours label Apr 25, 2023
@codebytere codebytere merged commit f4b7e59 into main Apr 25, 2023
17 checks passed
@codebytere codebytere deleted the handle-maybe-no-wc branch April 25, 2023 09:30
@release-clerk
Copy link

release-clerk bot commented Apr 25, 2023

Release Notes Persisted

Fixed a potential crash when right-clicking on macOS windows with draggable regions.

@trop
Copy link
Contributor

trop bot commented Apr 25, 2023

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

@trop trop bot added in-flight/25-x-y merged/25-x-y PR was merged to the "25-x-y" branch. and removed target/25-x-y PR should also be added to the "25-x-y" branch. in-flight/25-x-y labels Apr 25, 2023
@felixrieseberg
Copy link
Member

/trop run backport-to 24-x-y

@trop
Copy link
Contributor

trop bot commented Apr 28, 2023

@felixrieseberg is not authorized to run PR backports.

felixrieseberg pushed a commit to felixrieseberg/electron that referenced this pull request Apr 28, 2023
chore: fix crash on missing RenderWidgetHostView
@trop
Copy link
Contributor

trop bot commented Apr 28, 2023

@felixrieseberg has manually backported this PR to "24-x-y", please check out #38136

jkleinsc pushed a commit that referenced this pull request May 1, 2023
fix: crash on missing `RenderWidgetHostView` (#38100)

chore: fix crash on missing RenderWidgetHostView

Co-authored-by: Shelley Vohr <shelley.vohr@gmail.com>
@trop trop bot added the merged/24-x-y PR was merged to the "24-x-y" branch label May 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merged/24-x-y PR was merged to the "24-x-y" branch merged/25-x-y PR was merged to the "25-x-y" branch. semver/patch backwards-compatible bug fixes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants