Spams system logs with "Current memory footprint: 10 MB"

https://bugs.webkit.org/show_bug.cgi?id=213991

Patch by Michael Catanzaro <[email protected]> on 2020-07-06
Reviewed by Darin Adler.

This particular log statement is too much spam in the system journal, since it prints
continuously under normal operating conditions. Let's limit it only to Apple platforms,
which have had RELEASE_LOG() for a long time and where this is presumably expected.

* wtf/MemoryPressureHandler.cpp:
(WTF::MemoryPressureHandler::measurementTimerFired):

git-svn-id: http://svn.webkit.org/repository/webkit/trunk@263978 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Author: [email protected]

Source/WTF/ChangeLog

@@ -1,3 +1,17 @@
+2020-07-06  Michael Catanzaro  <[email protected]>
+
+        Spams system logs with "Current memory footprint: 10 MB"
+        https://bugs.webkit.org/show_bug.cgi?id=213991
+
+        Reviewed by Darin Adler.
+
+        This particular log statement is too much spam in the system journal, since it prints
+        continuously under normal operating conditions. Let's limit it only to Apple platforms,
+        which have had RELEASE_LOG() for a long time and where this is presumably expected.
+
+        * wtf/MemoryPressureHandler.cpp:
+        (WTF::MemoryPressureHandler::measurementTimerFired):
+
 2020-07-06  Sam Weinig  <[email protected]>
 
         Compile-time enable (but leave disabled at runtime by default) date/time input types on macOS to allow testing of cross platform (e.g. DOM) aspects of the controls

Source/WTF/wtf/MemoryPressureHandler.cpp

@@ -191,7 +191,9 @@ void MemoryPressureHandler::setMemoryUsagePolicyBasedOnFootprint(size_t footprin
 void MemoryPressureHandler::measurementTimerFired()
 {
     size_t footprint = memoryFootprint();
+#if PLATFORM(COCOA)
     RELEASE_LOG(MemoryPressure, "Current memory footprint: %zu MB", footprint / MB);
+#endif
     if (footprint >= thresholdForMemoryKill()) {
         shrinkOrDie();
         return;