1765278 - [META] Add support for Record and Tuples in DevTools

Status: ASSIGNED

(DevTools :: Console, task)

Description

[Nicolas Chevobbe [:nchevobbe]]

See the proposal: https://github.com/tc39/proposal-record-tuple

This is currently (partially?) implemented behind a config flag in SpiderMonkey (--enable-record-tuple)

There's probably a few things we need to do:

• Make sure the babel parser can consume scripts with record/tuples (as it introduces a new syntax)
• Check that pretty-print in the debugger isn't broken
• Check that beautifier in console editor isn't broken
• Add new Reps for Record and Tuple
• Ensure we can inspect those in DevTools

Comment 1

[Nicolas Chevobbe [:nchevobbe]]

Matthew, do you know approximately when we want to enable Records/Tuples for good?
I want to know if/when we should prioritize this so at least DevTools aren't broken when people try them in Firefox (at the moment, evaluating a record makes the tab crash)

Comment 2

[Matthew Gaudet (he/him) [:mgaudet]]

I'm not sure of the hoped-for timeline; given JIT support isn't done, certainly not for a release or two.

Regardless, crashes in devtools would block shipping. I tried to reproduce, but was blocked by Bug 1765359; would you be able to open a bug for the tab crash for us?

Comment 3

[Nicolas Chevobbe [:nchevobbe]]

(In reply to Matthew Gaudet (he/him) [:mgaudet] from comment #2)

I'm not sure of the hoped-for timeline; given JIT support isn't done, certainly not for a release or two.

Okay that's good to know, this gives us some time to add nice DevTools support

Regardless, crashes in devtools would block shipping. I tried to reproduce, but was blocked by Bug 1765359; would you be able to open a bug for the tab crash for us?

Sure, I opened Bug 1765477