Closed Bug 1667330 Opened 1 year ago Closed 1 year ago

Remove constructorName code

Categories

(Core :: JavaScript Engine: JIT, task, P1)

task

Tracking

()

RESOLVED FIXED
83 Branch
Tracking Status
firefox83 --- fixed

People

(Reporter: jandem, Assigned: jandem)

References

Details

Attachments

(1 file)

The getConstructorName testing function and some UbiNode code gets the constructor name for an object via the ObjectGroup's TypeNewScript. This doesn't work without TI and I think it was never a great idea because it depends on optimization heuristics.

This affects jsapi-tests with Warp enabled.

The getConstructorName testing function and UbiNode got the constructor name of a
JS object via the ObjectGroup's TypeNewScript. There's currently no way to get that
information without TI (and it always depended on brittle optimization heuristics)
so this patch removes that code.

Pushed by jdemooij@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/e2ed122bd760
Stop exposing constructor names of JS objects. r=iain
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 83 Branch
You need to log in before you can comment on or make changes to this bug.