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: Ensure electron delay loads the same modules as chromium #25428
Conversation
…ndows This change adds the same module delay load list that chromium uses for electron. Some modules were already getting delay loaded from other build files in chromium but not the main list via //build/config/win:delayloads. We do not include the list of delay loads in delayloads_not_for_child_dll as those have issues being loaded in sandboxes processes. This will reduce the overall reference set impact of the electron processes.
💖 Thanks for opening this pull request! 💖 We use semantic commit messages to streamline the release process. Before your pull request can be merged, you should update your pull request title to start with a semantic prefix. Examples of commit messages with semantic prefixes:
Things that will help get your PR across the finish line:
We get a lot of pull requests on this repo, so please be patient and we will get back to you as soon as we can. |
Not sure what the build-mac failures are. I can't access logs. Also, my changes shouldn't impact anything but Windows. |
mac failures seem to be flakes. |
@chrdavis For future reference you should be able to view the CI build log just by signing in, you have to auth with github but the logs aren't behind any kind of permission barrier. If you can't view them after signing in to CircleCI let me know and I can look into it 😄 |
Do the mac tests have to be run again or is this ok to merge now? |
It is fine to merge, the failures are unrelated. Thanks for the pr! |
Congrats on merging your first pull request! 🎉🎉🎉 |
Release Notes Persisted
|
I have automatically backported this PR to "10-x-y", please check out #25435 |
I have automatically backported this PR to "11-x-y", please check out #25436 |
I have automatically backported this PR to "9-x-y", please check out #25437 |
…on#25428) * Ensure electron delay loads the appropriate modules as chromium on windows This change adds the same module delay load list that chromium uses for electron. Some modules were already getting delay loaded from other build files in chromium but not the main list via //build/config/win:delayloads. We do not include the list of delay loads in delayloads_not_for_child_dll as those have issues being loaded in sandboxes processes. This will reduce the overall reference set impact of the electron processes. * fix: Ensure win modules are properly delayloaded * chore: fix linting Co-authored-by: Samuel Attard <samuel.r.attard@gmail.com>
This change adds the same module delay load list that chromium uses for electron. Some modules were already getting delay loaded from other build files in chromium but not the main list via //build/config/win:delayloads. We do not include the list of delay loads in delayloads_not_for_child_dll as those have issues being loaded in sandboxes processes. This will reduce the overall reference set impact of the electron processes.
notes: Added missing module delay loads on windows to reduce per process reference set impact