Closed
Bug 890913
Opened 11 years ago
Closed 9 years ago
Parser.jsm: support SpreadExpression
Categories
(DevTools :: Debugger, defect, P3)
DevTools
Debugger
Tracking
(firefox48 fixed)
RESOLVED
FIXED
Firefox 48
Tracking | Status | |
---|---|---|
firefox48 | --- | fixed |
People
(Reporter: past, Assigned: jsnajdr)
References
Details
Attachments
(1 file)
(deleted),
patch
|
vporof
:
review+
|
Details | Diff | Splinter Review |
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.
Updated•11 years ago
|
Priority: -- → P3
Updated•10 years ago
|
Summary: Support SpreadExpression in Parser.jsm → Parser.jsm: support SpreadExpression
Assignee | ||
Updated•9 years ago
|
Assignee: nobody → jsnajdr
Assignee | ||
Comment 1•9 years ago
|
||
- 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)
Assignee | ||
Comment 2•9 years ago
|
||
Updated•9 years ago
|
Attachment #8737735 -
Flags: review?(vporof) → review+
Assignee | ||
Comment 3•9 years ago
|
||
Checkin only after patches from bug 1260756, otherwise will cause conflicts.
Keywords: checkin-needed
Keywords: checkin-needed
Comment 5•9 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 9 years ago
status-firefox48:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 48
Updated•6 years ago
|
Product: Firefox → DevTools
You need to log in
before you can comment on or make changes to this bug.
Description
•