Skip to content
This repository has been archived by the owner on Apr 14, 2021. It is now read-only.

To close #2 #3

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

cool-Blue
Copy link

closes #2

Proposed solution is to wrap the worker's terminate method and revoke the blob object URL there.

@andyearnshaw
Copy link
Owner

Hey, thanks for the PR and apologies for not spotting it in the last few months. Would it be more appropriate to revoke the object URL after the worker has been initialised rather than doing it on terminate?

I'm thinking that many workers aren't terminated until the page unloads and, while the memory usage isn't a big deal I'd prefer to remain as efficient as possible.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Blob-based Workers fail to start in Edge
2 participants