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: don't handle browser messages before document element is created #19718
Conversation
DidCreateDocumentElement is called before the ElectronApiServiceImpl gets bound.
0ec005d
to
34a147e
Compare
A maintainer has manually backported this PR to "6-0-x", please check out #19719 |
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.
I'm not fluent in mojo but from a purely cpp standing this looks like what we want to be doing and the test case proves it 👍
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.
LGTM on the mojo changes.
Release Notes Persisted
|
I have automatically backported this PR to "7-0-x", please check out #19722 |
Description of Change
Close #19368.
This test was introduced by #1044 but then lost when migrating IPC module to mojo. This PR adds the fix back.
Checklist
npm test
passesRelease Notes
Notes: Fix async Node APIs not working after received IPC message.