Skip to content

Commit

Permalink
Merge pull request #984 from thewtex/revoke-pipeline
Browse files Browse the repository at this point in the history
fix(createWebWorkerPromise): revoke revokeObjectURL
  • Loading branch information
thewtex authored Oct 29, 2023
2 parents 16302b7 + 3084ebb commit c103bf4
Showing 1 changed file with 0 additions and 2 deletions.
2 changes: 0 additions & 2 deletions src/core/createWebWorkerPromise.ts
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,6 @@ async function createWebWorkerPromise (existingWorker: Worker | null, pipelineWo
const response = await axios.get(`${webWorkerString}/bundles/pipeline.${min}worker.js`, { responseType: 'blob' })
const workerObjectUrl = URL.createObjectURL(response.data as Blob)
worker = new Worker(workerObjectUrl, { type: 'module' })
URL.revokeObjectURL(workerObjectUrl)
} else {
worker = new Worker(`${webWorkerString}/bundles/pipeline.${min}worker.js`, { type: 'module' })
}
Expand All @@ -57,7 +56,6 @@ async function createWebWorkerPromise (existingWorker: Worker | null, pipelineWo
const response = await axios.get(workerUrl, { responseType: 'blob' })
const workerObjectUrl = URL.createObjectURL(response.data as Blob)
worker = new Worker(workerObjectUrl, { type: 'module' })
URL.revokeObjectURL(workerObjectUrl)
} else {
worker = new Worker(workerUrl, { type: 'module' })
}
Expand Down

0 comments on commit c103bf4

Please sign in to comment.