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

perf: pass primitives directly through the context bridge, avoids copying #24531

Merged
merged 1 commit into from
Jul 15, 2020

Conversation

MarshallOfSound
Copy link
Member

Primitives always use the prototypes of their accessing contexts, this means that we directly send primitives like strings and numbers over the bridge without copying. This reduces memory usage and improves performance as we don't have to serialize / deserialize needlessly.

Notes: Improved the performance of sending JS primitives over the context bridge

@electron-cation electron-cation bot added the new-pr 🌱 PR opened in the last 24 hours label Jul 13, 2020
@electron-cation electron-cation bot removed the new-pr 🌱 PR opened in the last 24 hours label Jul 14, 2020
@MarshallOfSound MarshallOfSound merged commit 3f54f24 into master Jul 15, 2020
@release-clerk
Copy link

release-clerk bot commented Jul 15, 2020

Release Notes Persisted

Improved the performance of sending JS primitives over the context bridge

@MarshallOfSound MarshallOfSound deleted the ctx-bridge-primtives branch July 15, 2020 01:38
@trop
Copy link
Contributor

trop bot commented Jul 15, 2020

I was unable to backport this PR to "9-x-y" cleanly;
you will need to perform this backport manually.

@trop
Copy link
Contributor

trop bot commented Jul 15, 2020

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

@trop
Copy link
Contributor

trop bot commented Jul 27, 2020

@MarshallOfSound has manually backported this PR to "9-x-y", please check out #24746

sentialx pushed a commit to sentialx/electron that referenced this pull request Jul 30, 2020
sentialx pushed a commit to sentialx/electron that referenced this pull request Apr 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants