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: destroy WebContents synchronously on shutdown (4-0-x) #15640

Merged
merged 1 commit into from Nov 8, 2018

Conversation

zcbenz
Copy link
Member

@zcbenz zcbenz commented Nov 8, 2018

Backport #15541 to 4-0-x.

Notes: Fix crash on exit when using BrowserView.

@zcbenz zcbenz requested review from a team November 8, 2018 12:39
@zcbenz zcbenz force-pushed the destroy-webcontents-sync-4-0-x branch from ae9f4ef to 7048a42 Compare November 8, 2018 12:41
@zcbenz zcbenz requested a review from a team November 8, 2018 12:41
Copy link
Contributor

@jkleinsc jkleinsc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@codebytere codebytere merged commit 8341445 into 4-0-x Nov 8, 2018
@release-clerk
Copy link

release-clerk bot commented Nov 8, 2018

Release Notes Persisted

Fix crash on exit when using BrowserView.

@codebytere codebytere deleted the destroy-webcontents-sync-4-0-x branch November 8, 2018 16:09
@miniak
Copy link
Contributor

miniak commented Mar 15, 2019

/trop run backport-to 3-1-x

@trop
Copy link
Contributor

trop bot commented Mar 15, 2019

The backport process for this PR has been manually initiated,
sending your 1's and 0's to "3-1-x" here we go! :D

@trop
Copy link
Contributor

trop bot commented Mar 15, 2019

I was unable to backport this PR to "3-1-x" cleanly;
you will need to perform this backport manually.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants