[jsdbg2] tenuring log is not used, and should be deleted (drainTenurePromotionsLog, etc)

RESOLVED FIXED in mozilla47

Status

()

defect
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: jimb, Assigned: jimb)

Tracking

unspecified
mozilla47
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

Assignee

Description

3 years ago
The Debugger API's tenure promotions log should be deleted.

The tenure promotions log is not used by the developer tools, and we have no immediate plans to use it. It's also difficult to make the log work consistently if we permit minor garbage collections to be triggered by the act of recording allocation metadata (call stacks) itself: the object gets tenured before we're able to attach a stack to it.
Assignee

Comment 1

3 years ago
Long Live The Tenure Promotions Log!
Attachment #8717708 - Flags: review?(nfitzgerald)
Comment on attachment 8717708 [details] [diff] [review]
Remove Debugger's tenure promotion log.

Review of attachment 8717708 [details] [diff] [review]:
-----------------------------------------------------------------

I don't think we strictly need JS::Zone::hasDebuggers nor js::Debugger::debuggeeZones anymore either (I added them for the tenure promotions log), but it seems like we might want to keep them around anyways for the brave new zone-focused, cx-free world.
Attachment #8717708 - Flags: review?(nfitzgerald) → review+
Comment on attachment 8717708 [details] [diff] [review]
Remove Debugger's tenure promotion log.

Review of attachment 8717708 [details] [diff] [review]:
-----------------------------------------------------------------

Also! Don't forget to remove the documentation for this API!
Assignee

Comment 5

3 years ago
Amended to delete the documentation as well. Carrying over r+.
Attachment #8717708 - Attachment is obsolete: true
Assignee

Updated

3 years ago
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Flags: in-testsuite+
OS: Unspecified → All
Hardware: Unspecified → All
Resolution: --- → FIXED
Target Milestone: --- → mozilla47
You need to log in before you can comment on or make changes to this bug.