The GC help threshold calculation was broken by the malloc memory tracking changes in bug 1395509 comment 33 to ignore the GC invocation kind.

This will need uplift to beta as it affects FF69.

This restores the use of GC invocation kind that was inadvertantly removed by the patch in bug 1395509 comment 33 from the call to ZoneAllocator::updateAllGCThresholds in GCRuntime::endSweepingSweepGroup. Also refactors ZoneMallocThreshold::computeZoneTriggerBytes a little.

Bugbug thinks this bug is a task, but please change it back in case of error.

https://hg.mozilla.org/mozilla-central/rev/84bdb98325f0

Comment on attachment 9076496 [details]
Bug 1564078 - Fix use of invocation kind that was previously lost and refactor malloc threshold updates r=sfink?

Beta/Release Uplift Approval Request

• User impact if declined: This fixes a possible memory size regression on beta.
• Is this code covered by automated tests?: Yes
• Has the fix been verified in Nightly?: Yes
• Needs manual test from QE?: No
• If yes, steps to reproduce:
• List of other uplifts needed: None
• Risk to taking this patch: Low
• Why is the change risky/not risky? (and alternatives if risky): This reverts the behaviour to its previous state before it was changed by bug 1395509.
• String changes made/needed: None.

Comment on attachment 9076496 [details]
Bug 1564078 - Fix use of invocation kind that was previously lost and refactor malloc threshold updates r=sfink?

Fixes a memory usage regression in 69. Approved for 69.0b4.

https://hg.mozilla.org/releases/mozilla-beta/rev/1bb8756f2af0