1575175 - Memory counters have gcBytes fields that are now used for both GC heap and malloc memory

Status: RESOLVED FIXED

(Core :: JavaScript: GC, task, P3)

Description

[Jon Coppeard (:jonco) (PTO until 14th September)]

Steve mentioned in bug 1574694 that we use the name gcBytes to apply to byte counts for both GC heap and malloc memory now. We could do some renaming to make things clearer.

Comment 1

[Jon Coppeard (:jonco) (PTO until 14th September)]

Attachment: Bug 1575175 - Rename memory counter classes now they're used for both GC and malloc heaps r?sfink

This renames:
HeapSize::gcBytes -> bytes (it's not just for GC heaps any more)
ZoneThreshold -> HeapThreshold (to go with HeapSize)
HeapThreshold::triggerBytes -> bytes (what else could it be?)

I renamed the ZoneAllocator members to make them more uniform/consitent so we now have gcHeapSize/gcHeapThreshold, mallocHeapSize/mallocHeapThreshold etc.

I also renamed the heap threshold classes.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/8a10768ff385
Rename memory counter classes now they're used for both GC and malloc heaps r=sfink

Comment 3

[Andrei Ciure[:aciure]]

https://hg.mozilla.org/mozilla-central/rev/8a10768ff385