Closed Bug 1290584 Opened 5 years ago Closed 5 years ago

Reassort stack marking code

Categories

(Core :: JavaScript: GC, defect)

Product:
Core
Component:
JavaScript: GC
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla51
Tracking Flags:
Tracking Status
firefox50 --- affected
firefox51 --- fixed

People

(Reporter: terrence, Assigned: terrence)

References

Details

Attachments

(1 file)

4_lift_stack_marking-v0.diff
4.69 KB, patch
jonco
: review+
Details | Diff | Splinter Review 
The way we've got this sorted right now just makes no sense whatsoever. This moves all of the C and JS stack marking under one reporter, which I thinks makes more sense than doing it by type, as we are right now. And of course, I've no idea what traceSelfHostingGlobal has to do with marking rooters. I'll move these under a different category later, once all the other non-shared bits are excised.
Attachment #8776143 - Flags: review?(jcoppeard)
Blocks: 1290591
Attachment #8776143 - Flags: review?(jcoppeard) → review+
https://hg.mozilla.org/mozilla-central/rev/f6efc93a3267
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
status-firefox51: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla51
You need to log in before you can comment on or make changes to this bug.