Rename legacy-iterator to property-iterator in jsiter.cpp

RESOLVED FIXED in Firefox 57

Status

()

enhancement
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: anba, Assigned: anba)

Tracking

Trunk
mozilla57
Points:
---

Firefox Tracking Flags

(firefox57 fixed)

Details

Attachments

(1 attachment)

Comment hidden (empty)
(Assignee)

Comment 1

2 years ago
builtin/Iterator.js:
- LegacyIterator() and LegacyIteratorShim() are no longer needed
- Also inlined NewLegacyIterator() into its sole caller (LegacyGeneratorIteratorShim)

jit-test/lib/range.js
- This file was only included, but its range() method was never called

jsfriendapi.h:
- Removed no longer used JSITER_KEYVALUE

jsiter.cpp:
- Renamed multiple methods to use "property iterator" instead of "legacy iterator" to reflect their current use

tests/ecma_N:
- And updated the tests in tests/ecma_N to no longer special-case __iterator__ resp. directly throw an error
Attachment #8906120 - Flags: review?(luke)

Updated

2 years ago
Attachment #8906120 - Flags: review?(luke) → review+

Comment 3

2 years ago
Pushed by ryanvm@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/e62cc9325a2b
Rename LegacyIterator to PropertyIterator in jsiter. r=luke
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/e62cc9325a2b
Status: ASSIGNED → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla57
You need to log in before you can comment on or make changes to this bug.