790589 - Add env variable to define the cc/gc log directory

Status: RESOLVED FIXED

(Core :: XPCOM, defect)

Description

[Olli Pettay [:smaug][[e-mail]]]

Patch coming.
This helps with mobile.

Comment 1

[Olli Pettay [:smaug][[e-mail]]]

Attachment: patch

Hopefully this is enough

Comment 2

[Olli Pettay [:smaug][[e-mail]]]

https://tbpl.mozilla.org/?tree=Try

Comment 3

[Olli Pettay [:smaug][[e-mail]]]

https://hg.mozilla.org/mozilla-central/rev/f36b93c70d05

Comment 4

[Andrew McCreight [:mccr8]]

Somebody added a note about this to the CC dump Wiki.  I fixed a typo in the environment variable name. ;)

https://wiki.mozilla.org/Performance:Leak_Tools#Cycle_collector_heap_dump

Comment 5

[David Baron :dbaron:]

Attachment: Document the MOZ_CC_LOG_DIRECTORY environment variable.

Comment 6

[Olli Pettay [:smaug][[e-mail]]]

Comment on attachment 715329 [details] [diff] [review]
Document the MOZ_CC_LOG_DIRECTORY environment variable.

maybe s/see/see for example/ because the most common case for logs is using
nsICycleCollectorListener and not using XPCOM_CC_LOG_ALL or XPCOM_CC_LOG_SHUTDOWN

Comment 7

[David Baron :dbaron:]

https://hg.mozilla.org/integration/mozilla-inbound/rev/8c46f89ed1a2

Any reason this bug wasn't marked fixed ages ago?

Comment 8

[Ryan VanderMeulen [:RyanVM]]

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

Comment 9

[Andrew McCreight [:mccr8]]

This looks fixed to me.