Skip to content
This repository was archived by the owner on Sep 9, 2021. It is now read-only.

Commit f729e34

Browse files
fix: memory leak for inline workers (#252)
1 parent f03498d commit f729e34

File tree

2 files changed

+8
-2
lines changed

2 files changed

+8
-2
lines changed

src/workers/InlineWorker.js

+6-1
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,12 @@ module.exports = function inlineWorker(content, url, workerType) {
3939
blob = new Blob([content]);
4040
}
4141

42-
return CreateWorker(URL.createObjectURL(blob), workerType);
42+
var objectURL = URL.createObjectURL(blob);
43+
var worker = CreateWorker(objectURL, workerType);
44+
45+
URL.revokeObjectURL(objectURL);
46+
47+
return worker;
4348
} catch (e) {
4449
return CreateWorker(
4550
'data:application/javascript,' + encodeURIComponent(content),

src/workers/index.js

+2-1
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,8 @@ const getWorker = (file, content, options) => {
2020
)}, ${fallbackWorkerPath}, ${options.workerType})`;
2121
}
2222

23-
let worker = 'Worker';
23+
let worker;
24+
2425
switch (options.workerType) {
2526
case 'SharedWorker':
2627
worker = 'SharedWorker';

0 commit comments

Comments
 (0)