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

feat: allow custom v8 snapshots to be used in the main process and the default snapshot in the renderer process #35694

Conversation

trop[bot]
Copy link
Contributor

@trop trop bot commented Sep 15, 2022

Backport of #35266

See that PR for details.

Notes: Added LoadBrowserProcessSpecificV8Snapshot as a new fuse that will let the main/browser process load its v8 snapshot from a file at browser_v8_context_snapshot.bin. Any other process will use the same path as is used today.

@trop trop bot requested review from a team as code owners September 15, 2022 20:21
@electron-cation electron-cation bot added the new-pr 🌱 PR opened in the last 24 hours label Sep 15, 2022
@trop trop bot added 20-x-y backport This is a backport PR backport/requested 🗳 semver/minor backwards-compatible functionality labels Sep 15, 2022
@electron-cation electron-cation bot removed the new-pr 🌱 PR opened in the last 24 hours label Sep 15, 2022
…e default snapshot in the renderer process (#35266)

* Updates to allow for using a custom v8 snapshot file name

* Allow using a custom v8 snapshot file name

* Fix up patch due to merge

* Use fuse to set up custom v8 snapshot file in browser process

* Refactor to use delegate instead of command line parameter

* Refactoring

* Update due to merge

* PR comments

* Rename patch

* Rename patch

chore: update patches
@jkleinsc jkleinsc force-pushed the trop/20-x-y-bp-feat-allow-custom-v8-snapshots-to-be-used-in-the-main-process-and-the-default-snapshot-in-the-renderer-process-1663273303061 branch from d776da2 to caa63db Compare September 19, 2022 14:25
Copy link
Member

@VerteDinde VerteDinde left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved on behalf of @electron/wg-releases

@VerteDinde VerteDinde merged commit 9e419bc into 20-x-y Sep 22, 2022
@VerteDinde VerteDinde deleted the trop/20-x-y-bp-feat-allow-custom-v8-snapshots-to-be-used-in-the-main-process-and-the-default-snapshot-in-the-renderer-process-1663273303061 branch September 22, 2022 16:41
@release-clerk
Copy link

release-clerk bot commented Sep 22, 2022

Release Notes Persisted

Added LoadBrowserProcessSpecificV8Snapshot as a new fuse that will let the main/browser process load its v8 snapshot from a file at browser_v8_context_snapshot.bin. Any other process will use the same path as is used today.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
20-x-y backport/approved ✅ backport This is a backport PR semver/minor backwards-compatible functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants