Closed Bug 1238779 Opened 9 years ago Closed 9 years ago

Remove the unused and nonsensical JS_TraceRuntime

Categories

(Core :: JavaScript: GC, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla46
Tracking Status
firefox46 --- fixed

People

(Reporter: terrence, Assigned: terrence)

References

(Blocks 2 open bugs)

Details

Attachments

(1 file)

Attached patch remove_JS_TraceRuntime-v0.diff (deleted) — Splinter Review
The only user is UbiNode, which can use the internal variant. As you well know, the API exposed by this is not really coherent with anything we're doing during GC, so at the very least it should not be public if it doesn't have to be.
Attachment #8706660 - Flags: review?(nfitzgerald)
Comment on attachment 8706660 [details] [diff] [review] remove_JS_TraceRuntime-v0.diff Review of attachment 8706660 [details] [diff] [review]: ----------------------------------------------------------------- Yep.
Attachment #8706660 - Flags: review?(nfitzgerald) → review+
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla46
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: