Created attachment 571519 [details] [diff] [review] patch This does as it says. It also makes it a little easier to add new fields without having to worry about column alignment.
Comment on attachment 571519 [details] [diff] [review] patch I had an idea for an improvement to this. I'll post a new patch soon.
Created attachment 571762 [details] [diff] [review] patch v2 This patch improves on the previous one by putting the MOZ_GCTIMER data into the Error Console (replacing the somewhat wrong data that was there before). While I was at it, I also fixed the problem we have where we incorrectly think that all GCs are full GCs. With the fix, we will no longer run the cycle collector after a compartmental GC. Gregor, do you have time to review this? Chris said his queue is pretty full.
and follow-up https://hg.mozilla.org/mozilla-central/rev/b5373c0d5a88
/me mumbles something about a patch to dom/ not getting a review from a DOM peer. May I ask what the GC numbers mean ?
Especially, is the "Total" the total time in ms?
(In reply to Olli Pettay [:smaug] from comment #6) > /me mumbles something about a patch to dom/ not getting a review from a DOM > peer. > > May I ask what the GC numbers mean ? Uh, sorry. I thought nsJSEnvironment.cpp was a special case. The numbers are in milliseconds. I can add units, since I guess it is a little confusing. Mark and Sweep are the time to do marking and sweeping. The remaining numbers mostly just break down the sweeping number. +Chu and -Chu tell you how many GC chunks were allocated or freed since the last GC.