Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 8310ec8b3200942d386437fba3f7280336ca9b42
      
https://github.com/WebKit/WebKit/commit/8310ec8b3200942d386437fba3f7280336ca9b42
  Author: Tetsuharu Ohzeki <[email protected]>
  Date:   2026-06-16 (Tue, 16 Jun 2026)

  Changed paths:
    A JSTests/stress/iterator-prototype-drop-bug315085.js
    A JSTests/stress/iterator-prototype-take-bug315085.js
    M Source/JavaScriptCore/builtins/JSIteratorPrototype.js

  Log Message:
  -----------
  [JSC] Update iterator built-ins to reflect the change of 
github:tc39/ecma262#3776
https://bugs.webkit.org/show_bug.cgi?id=315085

Reviewed by Yusuke Suzuki.

The change of https://github.com/tc39/ecma262/pull/3776 for iterator helper 
builtins are categorized into:

- Our implementations should be changed due to they should throw a `RangeError` 
corresponds to the user input value.
    - `Iterator.prototype.drop`
    - `Iterator.prototype.take`
- The semantics has been a bit changed but it's not related to our 
implementation actually.
    - `Iterator.prototype.filter`
    - `Iterator.prototype.find`
    - `Iterator.prototype.flatMap`
    - `Iterator.prototype.forEach`
    - `Iterator.prototype.map`
    - `Iterator.prototype.reduce`
    - `Iterator.prototype.some`

Tests: JSTests/stress/iterator-prototype-drop-bug315085.js
       JSTests/stress/iterator-prototype-take-bug315085.js

* JSTests/stress/iterator-prototype-drop-bug315085.js: Added.
    https://commits.webkit.org/284597@main implements `Iterator.prototype.drop`
    but no stress tests. This change adds a minimum testcase.
* JSTests/stress/iterator-prototype-take-bug315085.js: Added.
    https://commits.webkit.org/284597@main implements `Iterator.prototype.take`
    but no stress tests. This change adds a minimum testcase.
* Source/JavaScriptCore/builtins/JSIteratorPrototype.js:

Canonical link: https://commits.webkit.org/315276@main



To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications

Reply via email to