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: View reordering on re-addition to same parent #42116

Conversation

trop[bot]
Copy link
Contributor

@trop trop bot commented May 10, 2024

Backport of #42085

See that PR for details.

Notes: Fixed a crash when the same WebContentsView is added via addChildView multiple times.

@trop trop bot requested review from codebytere and a team May 10, 2024 08:16
@trop trop bot added 31-x-y backport This is a backport PR semver/patch backwards-compatible bug fixes labels May 10, 2024
Co-authored-by: Shelley Vohr <shelley.vohr@gmail.com>
@codebytere codebytere force-pushed the trop/31-x-y-bp-fix-view-reordering-on-re-addition-to-same-parent-1715329000377 branch from b06c50b to d6bdc99 Compare May 10, 2024 09:34
@codebytere
Copy link
Member

unrelated flaky test:

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

@codebytere codebytere merged commit 8d41e6e into 31-x-y May 13, 2024
12 of 13 checks passed
@codebytere codebytere deleted the trop/31-x-y-bp-fix-view-reordering-on-re-addition-to-same-parent-1715329000377 branch May 13, 2024 09:48
Copy link

release-clerk bot commented May 13, 2024

Release Notes Persisted

Fixed a crash when the same WebContentsView is added via addChildView multiple times.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
31-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