Bug 268272

Summary:	[JSC] DestructuringAssignmentTarget should be evaluated prior to calling [[Get]] / stepping iterator
Product:	WebKit	Reporter:	Alexey Shvayka <ashvayka>
Component:	JavaScriptCore	Assignee:	Alexey Shvayka <ashvayka>
Status:	RESOLVED FIXED
Severity:	Normal	CC:	webkit-bug-importer
Priority:	P2	Keywords:	InRadar
Version:	WebKit Nightly Build
Hardware:	All
OS:	All
See Also:	https://bugs.webkit.org/show_bug.cgi?id=268271
Bug Depends on:
Bug Blocks:	268026

Alexey Shvayka

Reported 2024-01-28 21:27:36 PST

Related test262 failures: * test/language/expressions/assignment/dstr/array-elem-iter-rtrn-close-err.js * test/language/expressions/assignment/dstr/array-elem-iter-rtrn-close-null.js * test/language/expressions/assignment/dstr/array-elem-iter-rtrn-close.js * test/language/expressions/assignment/dstr/array-elem-iter-thrw-close-err.js * test/language/expressions/assignment/dstr/array-elem-iter-thrw-close.js * test/language/expressions/assignment/dstr/array-elem-trlg-iter-list-rtrn-close-err.js * test/language/expressions/assignment/dstr/array-elem-trlg-iter-list-rtrn-close-null.js * test/language/expressions/assignment/dstr/array-elem-trlg-iter-list-rtrn-close.js * test/language/expressions/assignment/dstr/array-elem-trlg-iter-list-thrw-close-err.js * test/language/expressions/assignment/dstr/array-elem-trlg-iter-list-thrw-close.js * test/language/expressions/assignment/dstr/array-elem-trlg-iter-rest-rtrn-close-err.js * test/language/expressions/assignment/dstr/array-elem-trlg-iter-rest-rtrn-close-null.js * test/language/expressions/assignment/dstr/array-elem-trlg-iter-rest-rtrn-close.js * test/language/expressions/assignment/dstr/array-elem-trlg-iter-rest-thrw-close-err.js * test/language/expressions/assignment/dstr/array-elem-trlg-iter-rest-thrw-close.js * test/language/expressions/assignment/dstr/array-rest-iter-rtrn-close-err.js * test/language/expressions/assignment/dstr/array-rest-iter-rtrn-close-null.js * test/language/expressions/assignment/dstr/array-rest-iter-rtrn-close.js * test/language/expressions/assignment/dstr/array-rest-iter-thrw-close-err.js * test/language/expressions/assignment/dstr/array-rest-iter-thrw-close.js * test/language/expressions/assignment/dstr/array-rest-lref-err.js * test/language/statements/for-of/dstr/array-elem-iter-rtrn-close-err.js * test/language/statements/for-of/dstr/array-elem-iter-rtrn-close-null.js * test/language/statements/for-of/dstr/array-elem-iter-rtrn-close.js * test/language/statements/for-of/dstr/array-elem-iter-thrw-close-err.js * test/language/statements/for-of/dstr/array-elem-iter-thrw-close.js * test/language/statements/for-of/dstr/array-elem-trlg-iter-list-rtrn-close-err.js * test/language/statements/for-of/dstr/array-elem-trlg-iter-list-rtrn-close-null.js * test/language/statements/for-of/dstr/array-elem-trlg-iter-list-rtrn-close.js * test/language/statements/for-of/dstr/array-elem-trlg-iter-list-thrw-close-err.js * test/language/statements/for-of/dstr/array-elem-trlg-iter-list-thrw-close.js * test/language/statements/for-of/dstr/array-elem-trlg-iter-rest-rtrn-close-err.js * test/language/statements/for-of/dstr/array-elem-trlg-iter-rest-rtrn-close-null.js * test/language/statements/for-of/dstr/array-elem-trlg-iter-rest-rtrn-close.js * test/language/statements/for-of/dstr/array-elem-trlg-iter-rest-thrw-close-err.js * test/language/statements/for-of/dstr/array-elem-trlg-iter-rest-thrw-close.js * test/language/statements/for-of/dstr/array-rest-iter-rtrn-close-err.js * test/language/statements/for-of/dstr/array-rest-iter-rtrn-close-null.js * test/language/statements/for-of/dstr/array-rest-iter-rtrn-close.js * test/language/statements/for-of/dstr/array-rest-iter-thrw-close-err.js * test/language/statements/for-of/dstr/array-rest-iter-thrw-close.js * test/language/statements/for-of/dstr/array-rest-lref-err.js

Attachments
Add attachment proposed patch, testcase, etc.

Alexey Shvayka

Comment 1 2024-01-28 21:31:09 PST

Spec: https://tc39.es/ecma262/#sec-runtime-semantics-iteratordestructuringassignmentevaluation Production: AssignmentElement : DestructuringAssignmentTarget Initializer[opt] Step: 1.a

Radar WebKit Bug Importer

Comment 2 2024-01-30 14:39:26 PST

<rdar://problem/121960976>

Alexey Shvayka

Comment 3 2024-07-01 17:12:44 PDT

Pull request: https://github.com/WebKit/WebKit/pull/30369

EWS

Comment 4 2024-07-16 12:19:02 PDT

Committed 281013@main (e14851d085d3): <https://commits.webkit.org/281013@main> Reviewed commits have been landed. Closing PR #30369 and removing active labels.

Note You need to log in before you can comment on or make changes to this bug.