965141 - Support ES6 syntax for prettyprinting

Status: RESOLVED FIXED

(DevTools :: Debugger, defect, P3)

Description

[Victor Porof [:vporof][:vp]]

for (let foo of bar) {}

Pretty prints to:

for (letfooofbar) {
}

Alternatively, if using var:

for (var fooofbar) {
}

Comment 1

[Nick Fitzgerald [:fitzgen] [⏰PST; UTC-8]]

Expected as it doesn't do ES6.

Would be nice if we could disable auto pretty printing only for chrome debugging.

Comment 3

[Brian Grinstead [:bgrins]]

We can pass `{ ecmaVersion: 6 }` as an option to the tokenizer and get this functionality (most likely would want to bump to the latest version of acorn to get more features supported, but I believe it should work with the current version).

Comment 4

[Brian Grinstead [:bgrins]]

There's a particular case of es6 prettyprining that seems to be fixed by passing the option to acorn in Bug 951622

Comment 5

[Brian Grinstead [:bgrins]]

...However, it doesn't fix the 
for (let foo of bar) -> for (let fooofbar) thing

Comment 6

[J. Ryan Stinnett [:jryans] (Use needinfo, replies may be slow)]

Bug 938203 seems to be a similar meta bug for the same issue, but with different bugs under it.  Someone may want to fold them together if they work on this issue.

Comment 7

[Kuba Niewiarowski]

Looks like duplicate of bug 938203...

Comment 8

[David Walsh :davidwalsh]

This is no longer an issue in the debugger.