allow null and undefined as the second argument of `structuredClo…

…ne`, close #1056
zloirock · Mar 14, 2022 · 887850a · 887850a
1 parent fb58217
commit 887850a
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,5 +1,6 @@
 ## Changelog
 ##### Unreleased
+- `null` and `undefined` allowed as the second argument of `structuredClone`, [#1056](https://github.com/zloirock/core-js/issues/1056)
 - Updated Electron 18.0 compat data mapping
 
 ##### 3.21.1 - 2022.02.17

diff --git a/packages/core-js/modules/web.structured-clone.js b/packages/core-js/modules/web.structured-clone.js
@@ -448,7 +448,7 @@ var tryToTransfer = function (rawTransfer, map) {
 
 $({ global: true, enumerable: true, sham: !PROPER_TRANSFER, forced: FORCED_REPLACEMENT }, {
   structuredClone: function structuredClone(value /* , { transfer } */) {
-    var options = validateArgumentsLength(arguments.length, 1) > 1 ? anObject(arguments[1]) : undefined;
+    var options = validateArgumentsLength(arguments.length, 1) > 1 && arguments[1] != null ? anObject(arguments[1]) : undefined;
     var transfer = options ? options.transfer : undefined;
     var map;
 

diff --git a/tests/pure/web.structured-clone.js b/tests/pure/web.structured-clone.js
@@ -21,6 +21,8 @@ QUnit.module('structuredClone', () => {
     assert.name(structuredClone, 'structuredClone');
     assert.arity(structuredClone, 1);
     assert.throws(() => structuredClone(), 'throws without arguments');
+    assert.same(structuredClone(1, null), 1, 'null as options');
+    assert.same(structuredClone(1, undefined), 1, 'undefined as options');
   });
 
   function cloneTest(value, verifyFunc) {

diff --git a/tests/tests/web.structured-clone.js b/tests/tests/web.structured-clone.js
@@ -13,6 +13,8 @@ QUnit.module('structuredClone', () => {
     assert.arity(structuredClone, 1);
     if (!NODE) assert.looksNative(structuredClone);
     assert.throws(() => structuredClone(), 'throws without arguments');
+    assert.same(structuredClone(1, null), 1, 'null as options');
+    assert.same(structuredClone(1, undefined), 1, 'undefined as options');
   });
 
   function cloneTest(value, verifyFunc) {