Enable MemoryPressureHandler for MALLOC_HEAP_BREAKDOWN

asurdej-comcast · pgorszkowski-igalia · commit ac6b407929a1 · 2024-05-02T08:14:15.000+02:00
Enabling MALLOC_HEAP_BREAKDOWN currently disables MemoryPressureHandler.
We would like to keep both enabled to be able to track
memory related issues.
diff --git a/Source/WTF/wtf/MemoryPressureHandler.cpp b/Source/WTF/wtf/MemoryPressureHandler.cpp
@@ -159,11 +159,13 @@ void MemoryPressureHandler::setMemoryFootprintPollIntervalForTesting(Seconds pol
 
 void MemoryPressureHandler::setShouldUsePeriodicMemoryMonitor(bool use)
 {
+#if !ENABLE(MALLOC_HEAP_BREAKDOWN)
     if (!isFastMallocEnabled()) {
         // If we're running with FastMalloc disabled, some kind of testing or debugging is probably happening.
         // Let's be nice and not enable the memory kill mechanism.
         return;
     }
+#endif
 
     if (use) {
         m_measurementTimer = makeUnique<RunLoop::Timer>(RunLoop::main(), this, &MemoryPressureHandler::measurementTimerFired);

Original file line number	Diff line number	Diff line change
`@@ -159,11 +159,13 @@ void MemoryPressureHandler::setMemoryFootprintPollIntervalForTesting(Seconds pol`
`159`	`159`
`160`	`160`	`void MemoryPressureHandler::setShouldUsePeriodicMemoryMonitor(bool use)`
`161`	`161`	`{`
	`162`	`+#if !ENABLE(MALLOC_HEAP_BREAKDOWN)`
`162`	`163`	`if (!isFastMallocEnabled()) {`
`163`	`164`	`// If we're running with FastMalloc disabled, some kind of testing or debugging is probably happening.`
`164`	`165`	`// Let's be nice and not enable the memory kill mechanism.`
`165`	`166`	`return;`
`166`	`167`	`}`
	`168`	`+#endif`
`167`	`169`
`168`	`170`	`if (use) {`
`169`	`171`	`m_measurementTimer = makeUnique<RunLoop::Timer>(RunLoop::main(), this, &MemoryPressureHandler::measurementTimerFired);`