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
Revoke blob URLs after the module is loaded #124
Conversation
src/es-module-shims.js
Outdated
function revokeObjectURLs(registryKeys) { | ||
function cleanup() { | ||
for (const key of registryKeys) { | ||
const blobURL = registry[key]?.b; |
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.
I'm not sure if we have optional chaining for early modules implementations...
69f19b9
to
59e43a8
Compare
59e43a8
to
de4e60c
Compare
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.
I really like this scheduling technique. We could possibly even apply this to the ESM analysis itself to break up jank during the "instantiate" phase.
@guybedford ha, good point actually, I can maybe look into that later 👍 |
No description provided.