802007 - Remove unneeded Mac-only memory reporters.

Status: RESOLVED FIXED

(Core :: XPCOM, defect)

Description

[Nicholas Nethercote [inactive]]

Back when we didn't use jemalloc on Mac, it was useful to have 
Mac-specific "heap-allocated" and "heap-unused" memory reporters.  But now 
they're dead code unless you're in a --disable-jemalloc build (along with
the "zone0" reporters), and they are a hindrance to DMD builds.

Comment 1

[Nicholas Nethercote [inactive]]

Attachment: Remove unneeded Mac-only memory reporters.

This patch removes the rarely-used Mac-specific memory reporters.  This
allows us to also remove HAVE_HEAP_ALLOCATED_REPORTERS and
HAVE_EXPLICIT_REPORTER, and just use HAVE_JEMALLOC_STATS instead.

Comment 2

[Justin Lebar (not reading bugmail)]

Comment on attachment 671725 [details] [diff] [review]
Remove unneeded Mac-only memory reporters.

Did we ever un-support 10.5?  We don't have jemalloc there (because of OS bugs).

http://www.mozilla.org/en-US/firefox/system-requirements.html still shows 10.5.

That's my only concern here.  r=me if we don't have to worry about 10.5.

Comment 3

[Justin Lebar (not reading bugmail)]

lsblakk tells me we unsupported 10.5 in FF17, so we're good here.

Comment 4

[Nicholas Nethercote [inactive]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/3dc836fcff43

Comment 5

[(no longer active)]

https://hg.mozilla.org/mozilla-central/rev/3dc836fcff43