-
Notifications
You must be signed in to change notification settings - Fork 15.1k
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: quit after Chromium is fully started #21488
Conversation
fffa366
to
8609c38
Compare
53e9030
to
8609c38
Compare
8609c38
to
5810321
Compare
The |
No Release Notes |
I have automatically backported this PR to "7-1-x", please check out #21505 |
I have automatically backported this PR to "6-1-x", please check out #21506 |
I have automatically backported this PR to "8-x-y", please check out #21507 |
Description of Change
Flush pending tasks before quitting Electron.
This is to fix crash on exit which happens when Electron quits while there are still some initialization work pending in message loop.
This should be able to fix some flaky tests on CI.
This does not completely fix the flaky tests that destroy WebContents, which I believe have other reasons for crashing.
Release Notes
Notes: no-notes