Parser.jsm: support SpreadExpression

RESOLVED FIXED in Firefox 48

Status

()

Firefox
Developer Tools: Debugger
P3
normal
RESOLVED FIXED
5 years ago
2 years ago

People

(Reporter: past, Assigned: jsnajdr)

Tracking

unspecified
Firefox 48
Points:
---

Firefox Tracking Flags

(firefox48 fixed)

Details

Attachments

(1 attachment)

Bug 834756 adds a workaround to avoid errors in case a SpreadExpression appears while parsing. We should add proper support for this feature and remove the workaround.
Priority: -- → P3

Updated

4 years ago
Summary: Support SpreadExpression in Parser.jsm → Parser.jsm: support SpreadExpression
(Assignee)

Updated

2 years ago
Assignee: nobody → jsnajdr
(Assignee)

Comment 1

2 years ago
Created attachment 8737735 [details] [diff] [review]
Parser.jsm: support SpreadExpression

- added support for SpreadExpression to Parser.jsm
- removed workaround code from ArrayExpression handler
- this patch needs to be applied on top of the patches from bug 1260756 (mainly because of the eslint cleanup)
Attachment #8737735 - Flags: review?(vporof)
Attachment #8737735 - Flags: review?(vporof) → review+
(Assignee)

Comment 3

2 years ago
Checkin only after patches from bug 1260756, otherwise will cause conflicts.
Keywords: checkin-needed

Comment 5

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/d86141a0a68a
Status: NEW → RESOLVED
Last Resolved: 2 years ago
status-firefox48: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 48
You need to log in before you can comment on or make changes to this bug.