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
feat: make ipc-message and ipc-message-sync events public #16468
Conversation
ipc-message
and ipc-message-sync
events public0c43c6e
to
d29e22b
Compare
fe38ad6
to
e5b6474
Compare
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.
Can you remove all refactors of the internal implementation of IPC. The only things that need to happen this PR
- Publicly document the events
- Change the signiture of
ipc-message*
events to beevent, channel, ...args
instead ofevent, [channel, args]
f19fda8
to
6828ed5
Compare
@MarshallOfSound done |
6828ed5
to
5b4e391
Compare
The CI is failing on 2 tests:
They don't seem to be flaky as they are failing on all platforms, though I'm not sure why this change would lead to the failures. |
5b4e391
to
c7dbed3
Compare
@zcbenz fixed |
c7dbed3
to
9afe998
Compare
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.
Nice!
Release Notes Persisted
|
Description of Change
webContents
.ipc-message
andipc-message-sync
onwebContents
are refactored to make the arguments easier to handle./cc @MarshallOfSound, @nornagon
Checklist
npm test
passesRelease Notes
Notes: Added
ipc-message
andipc-message-sync
events towebContents
.