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: "Failed to serialize arguments" when emitting 'context-menu' for webview #31259
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.
Can we instead make frame
a non enumerable getter
46196ed
to
84f680b
Compare
84f680b
to
96b8337
Compare
Release Notes Persisted
|
I have automatically backported this PR to "16-x-y", please check out #31278 |
I have automatically backported this PR to "15-x-y", please check out #31279 |
… webview (#31259) (#31279) * fix: sanitize params for 'context-menu' event sent over IPC for webview * Revert "fix: sanitize params for 'context-menu' event sent over IPC for webview" This reverts commit 7fee455. * fix: make frame property non-enumerable in params for 'context-menu' event Co-authored-by: Milan Burda <milan.burda@gmail.com>
… webview (electron#31259) * fix: sanitize params for 'context-menu' event sent over IPC for webview * Revert "fix: sanitize params for 'context-menu' event sent over IPC for webview" This reverts commit 7fee455. * fix: make frame property non-enumerable in params for 'context-menu' event
… webview (electron#31259) * fix: sanitize params for 'context-menu' event sent over IPC for webview * Revert "fix: sanitize params for 'context-menu' event sent over IPC for webview" This reverts commit 7fee455. * fix: make frame property non-enumerable in params for 'context-menu' event
… webview (electron#31259) * fix: sanitize params for 'context-menu' event sent over IPC for webview * Revert "fix: sanitize params for 'context-menu' event sent over IPC for webview" This reverts commit 7fee455. * fix: make frame property non-enumerable in params for 'context-menu' event
Description of Change
Fixes #31249 regressed by #30831
Checklist
npm test
passesRelease Notes
Notes: Fixed "Failed to serialize arguments" error when emitting 'context-menu' event for webview.