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: potential async_hooks crash in NotifyWindowRestore on Windows #41146

Conversation

trop[bot]
Copy link
Contributor

@trop trop bot commented Jan 26, 2024

Backport of #40576

See that PR for details.

Notes: Fixed a potential async_hooks crash when listening for the restore event on Windows after minimizing a maximized BrowserWindow.

trop bot and others added 2 commits January 26, 2024 18:53
Co-authored-by: Shelley Vohr <shelley.vohr@gmail.com>
Co-authored-by: Shelley Vohr <shelley.vohr@gmail.com>
@trop trop bot requested a review from codebytere January 26, 2024 18:54
@trop trop bot added 29-x-y backport This is a backport PR semver/patch backwards-compatible bug fixes labels Jan 26, 2024
@codebytere
Copy link
Member

Failing test:

BrowserWindow module focus and visibility BrowserWindow.focus() it does not activate the app if focusing an inactive panel

not related and present/flaky in other PRs.

@codebytere codebytere merged commit 25f52d5 into 29-x-y Jan 29, 2024
19 of 20 checks passed
@codebytere codebytere deleted the trop/29-x-y-bp-fix-potential-async_hooks-crash-in-notifywindowrestore-on-windows-1706295206087 branch January 29, 2024 09:11
Copy link

release-clerk bot commented Jan 29, 2024

Release Notes Persisted

Fixed a potential async_hooks crash when listening for the restore event on Windows after minimizing a maximized BrowserWindow.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
29-x-y backport This is a backport PR semver/patch backwards-compatible bug fixes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant