Skip to content

Commit 558a802

Browse files
committed
Removing the definition of the obsolete FinalizationGroup.
1 parent 351bf45 commit 558a802

File tree

2 files changed

+3
-8
lines changed

2 files changed

+3
-8
lines changed

graal-nodejs/lib/internal/bootstrap/node.js

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -55,14 +55,6 @@ setupProcessObject();
5555
if (typeof Packages !== 'undefined') {
5656
Packages[Symbol.toStringTag] = 'Packages'; // breaking isRhino check of acorn package
5757
}
58-
if (typeof FinalizationRegistry !== 'undefined') {
59-
Object.defineProperty(globalThis, 'FinalizationGroup', {
60-
value: FinalizationRegistry,
61-
enumerable: false,
62-
writable: true,
63-
configurable: true
64-
});
65-
}
6658
// graal-node.js patch end
6759

6860
setupGlobalProxy();

graal-nodejs/test/graal/unit/other.js

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -91,4 +91,7 @@ describe('Other', function () {
9191
// should not fail (fails when Node.js is built without intl support)
9292
new util.TextDecoder('utf-8', { fatal: true });
9393
});
94+
it('should not define FinalizationGroup', function () {
95+
assert.strictEqual(global.FinalizationGroup, undefined);
96+
});
9497
});

0 commit comments

Comments
 (0)