add (Async)Iterator.prototype.constructor -> (Async)Iterator

per tc39/proposal-iterator-helpers#60
zloirock · Oct 27, 2019 · 16cb899 · 16cb899
1 parent 22626ed
commit 16cb899
Show file tree

Hide file tree

Showing 7 changed files with 25 additions and 0 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -2,6 +2,7 @@
 ##### Unreleased
 - Added a workaround of V8 deoptimization which causes serious performance degradation (~4x in my tests) of `Array#concat`, [#679](https://github.com/zloirock/core-js/issues/679)
 - Added a workaround of V8 deoptimization which causes slightly performance degradation of `Promise`, [#679](https://github.com/zloirock/core-js/issues/679)
+- Added `(Async)Iterator.prototype.constructor -> (Async)Iterator` per [this issue](https://github.com/tc39/proposal-iterator-helpers/issues/60)
 - Added compat data for Chromium-based Edge
 
 ##### 3.3.4 - 2019.10.25

diff --git a/packages/core-js/modules/esnext.async-iterator.constructor.js b/packages/core-js/modules/esnext.async-iterator.constructor.js
@@ -20,6 +20,10 @@ if (!has(AsyncIteratorPrototype, TO_STRING_TAG)) {
   createNonEnumerableProperty(AsyncIteratorPrototype, TO_STRING_TAG, 'AsyncIterator');
 }
 
+if (!has(AsyncIteratorPrototype, 'constructor') || AsyncIteratorPrototype.constructor === Object) {
+  createNonEnumerableProperty(AsyncIteratorPrototype, 'constructor', AsyncIteratorConstructor);
+}
+
 $({ global: true, forced: IS_PURE }, {
   AsyncIterator: AsyncIteratorConstructor
 });
diff --git a/packages/core-js/modules/esnext.iterator.constructor.js b/packages/core-js/modules/esnext.iterator.constructor.js
@@ -32,6 +32,10 @@ if (!has(IteratorPrototype, TO_STRING_TAG)) {
   createNonEnumerableProperty(IteratorPrototype, TO_STRING_TAG, 'Iterator');
 }
 
+if (!has(IteratorPrototype, 'constructor') || IteratorPrototype.constructor === Object) {
+  createNonEnumerableProperty(IteratorPrototype, 'constructor', IteratorConstructor);
+}
+
 IteratorConstructor.prototype = IteratorPrototype;
 
 $({ global: true, forced: FORCED }, {

diff --git a/tests/pure/esnext.async-iterator.constructor.js b/tests/pure/esnext.async-iterator.constructor.js
@@ -12,6 +12,10 @@ QUnit.test('AsyncIterator', assert => {
   assert.throws(() => AsyncIterator(), 'throws w/o `new`');
 });
 
+QUnit.test('AsyncIterator#constructor', assert => {
+  assert.strictEqual(AsyncIterator.prototype.constructor, AsyncIterator, 'AsyncIterator#constructor is AsyncIterator');
+});
+
 QUnit.test('AsyncIterator#@@toStringTag', assert => {
   assert.strictEqual(AsyncIterator.prototype[Symbol.toStringTag], 'AsyncIterator', 'AsyncIterator::@@toStringTag is `AsyncIterator`');
 });
diff --git a/tests/pure/esnext.iterator.constructor.js b/tests/pure/esnext.iterator.constructor.js
@@ -13,6 +13,10 @@ QUnit.test('Iterator', assert => {
   assert.throws(() => Iterator(), 'throws w/o `new`');
 });
 
+QUnit.test('Iterator#constructor', assert => {
+  assert.strictEqual(Iterator.prototype.constructor, Iterator, 'Iterator#constructor is Iterator');
+});
+
 QUnit.test('Iterator#@@toStringTag', assert => {
   assert.strictEqual(Iterator.prototype[Symbol.toStringTag], 'Iterator', 'Iterator::@@toStringTag is `Iterator`');
 });
diff --git a/tests/tests/esnext.async-iterator.constructor.js b/tests/tests/esnext.async-iterator.constructor.js
@@ -19,6 +19,10 @@ QUnit.test('AsyncIterator', assert => {
   assert.throws(() => AsyncIterator(), 'throws w/o `new`');
 });
 
+QUnit.test('AsyncIterator#constructor', assert => {
+  assert.strictEqual(AsyncIterator.prototype.constructor, AsyncIterator, 'AsyncIterator#constructor is AsyncIterator');
+});
+
 QUnit.test('AsyncIterator#@@toStringTag', assert => {
   assert.strictEqual(AsyncIterator.prototype[Symbol.toStringTag], 'AsyncIterator', 'AsyncIterator::@@toStringTag is `AsyncIterator`');
   assert.strictEqual(String(AsyncIterator.from([1, 2, 3])), '[object AsyncIterator]', 'correct stringification');

diff --git a/tests/tests/esnext.iterator.constructor.js b/tests/tests/esnext.iterator.constructor.js
@@ -25,6 +25,10 @@ QUnit.test('Iterator', assert => {
   assert.throws(() => Iterator(), 'throws w/o `new`');
 });
 
+QUnit.test('Iterator#constructor', assert => {
+  assert.strictEqual(Iterator.prototype.constructor, Iterator, 'Iterator#constructor is AsyncIterator');
+});
+
 QUnit.test('Iterator#@@toStringTag', assert => {
   assert.strictEqual(Iterator.prototype[Symbol.toStringTag], 'Iterator', 'Iterator::@@toStringTag is `Iterator`');
   assert.strictEqual(String(Iterator.from({