Skip to content
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

Merged
merged 3 commits into from Oct 4, 2021

Conversation

miniak
Copy link
Contributor

@miniak miniak commented Oct 3, 2021

Description of Change

Fixes #31249 regressed by #30831
Screen Shot 2021-10-04 at 6 52 39 PM

Checklist

Release Notes

Notes: Fixed "Failed to serialize arguments" error when emitting 'context-menu' event for webview.

Copy link
Member

@MarshallOfSound MarshallOfSound left a comment

Can we instead make frame a non enumerable getter

@miniak miniak force-pushed the miniak/webview-context-menu branch from 46196ed to 84f680b Oct 4, 2021
@miniak miniak requested a review from MarshallOfSound Oct 4, 2021
@miniak miniak changed the title fix: sanitize params for 'context-menu' event sent over IPC for webview fix: "Failed to serialize arguments" when emitting 'context-menu' for webview Oct 4, 2021
@miniak miniak force-pushed the miniak/webview-context-menu branch from 84f680b to 96b8337 Oct 4, 2021
@miniak miniak requested a review from samuelmaddock Oct 4, 2021
@VerteDinde VerteDinde merged commit d7a7c12 into main Oct 4, 2021
17 checks passed
@VerteDinde VerteDinde deleted the miniak/webview-context-menu branch Oct 4, 2021
@release-clerk
Copy link

@release-clerk release-clerk bot commented Oct 4, 2021

Release Notes Persisted

Fixed "Failed to serialize arguments" error when emitting 'context-menu' event for webview.

trop bot pushed a commit that referenced this issue Oct 4, 2021
… webview (#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
@trop
Copy link
Contributor

@trop trop bot commented Oct 4, 2021

I have automatically backported this PR to "16-x-y", please check out #31278

@trop
Copy link
Contributor

@trop trop bot commented Oct 4, 2021

I have automatically backported this PR to "15-x-y", please check out #31279

@trop trop bot added the merged/15-x-y label Oct 4, 2021
VerteDinde pushed a commit that referenced this issue Oct 4, 2021
… 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>
t57ser added a commit to t57ser/electron that referenced this issue Oct 27, 2021
… 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
t57ser added a commit to t57ser/electron that referenced this issue Oct 27, 2021
… 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
t57ser added a commit to t57ser/electron that referenced this issue Oct 29, 2021
… 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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment