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: ensure ready-to-show event is fired #25448
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@zcbenz looks like this is causing some test failures
It seems that when loading |
Release Notes Persisted
|
I was unable to backport this PR to "10-x-y" cleanly; |
I have automatically backported this PR to "11-x-y", please check out #25471 |
I was unable to backport this PR to "9-x-y" cleanly; |
I was unable to backport this PR to "8-x-y" cleanly; |
* fix: ensure ready-to-show event is fired * test: acutally draw something when capturePage
This reverts commit e5933c6.
This reverts commit e5933c6.
* some time ago @electron made "ready-to-show" event to be fired multiple times which goes against their documentation (possibly caused by electron/electron#25448) and so was causing the annoying windows flickering in the app
* some time ago @electron made "ready-to-show" event to be fired multiple times which goes against their documentation (possibly caused by electron/electron#25448) and so was causing the annoying windows flickering in the app
Description of Change
Close #25253.
The
ready-to-show
event is not very stable that, sometimes on some machines it might not be fired, and the actual behavior depends on the version of Chromium.To work around this bug, we ensure the
ready-to-show
event is emitted if it has not been emitted afterdid-finish-load
event.The real fix should be fixing the event itself, but unfortunately we haven't been able to find out why it did not emit so far, you can check #7779 on the history of this bug.
Checklist
npm test
passesRelease Notes
Notes: Fix
ready-to-show
event not emitted on some machines.