1415188 - Consider removing Reflect.parse's "builder" API

Status: RESOLVED FIXED

(Core :: JavaScript Engine, enhancement, P5)

Description

[Jason Orendorff [:jorendorff]]

Reflect.parse takes an optional `builder` argument. If present, it's an object with a ton of methods, and it's used to build the resulting AST. (The default, if `builder` is missing, is to build a straightforward JSON AST.)

https://developer.mozilla.org/en-US/docs/Mozilla/Projects/SpiderMonkey/Parser_API#Builder_objects

This API is brittle. Every time we add anything to JS syntax--or remove an obsolete feature--all code using the builder API breaks. Therefore I doubt anyone is using it and we should remove it.

Comment 1

[Firefox Bug Husbandry Bot]

https://wiki.mozilla.org/Bug_Triage/Projects/Bug_Handling/Bug_Husbandry#Move_fix-optionals

Comment 2

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

This seems to be done: All we check these days is loc and target.

Comment 3

[Iain Ireland [:iain]]

This was done in bug 1800629.