Closed
Bug 1667330
Opened 4 years ago
Closed 4 years ago
Remove constructorName code
Categories
(Core :: JavaScript Engine: JIT, task, P1)
Core
JavaScript Engine: JIT
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.
|
Assignee | |
Comment 1•4 years ago
|
||
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
|
||
Comment 3•4 years ago
|
||
| bugherder | ||
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
status-firefox83:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 83 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•