From 36830a2264cd29cc45dda3ebd95e60da9ebbf3bc Mon Sep 17 00:00:00 2001
From: Gavin Corkery <gavincorkery@google.com>
Date: Mon, 10 Aug 2020 11:08:00 +0100
Subject: Only log health check state changes

Reduce log spam from PackageWatchdog by only logging when the
health check state of a package changes.

Test: adb logcat
Bug: 162623584
Change-Id: I7c270e503119785f3c18c042adeb53f0ccc913a7
---
 services/core/java/com/android/server/PackageWatchdog.java | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/services/core/java/com/android/server/PackageWatchdog.java b/services/core/java/com/android/server/PackageWatchdog.java
index 1689656479a5..e675d8d458c7 100644
--- a/services/core/java/com/android/server/PackageWatchdog.java
+++ b/services/core/java/com/android/server/PackageWatchdog.java
@@ -1453,8 +1453,11 @@ public class PackageWatchdog {
             } else {
                 mHealthCheckState = HealthCheckState.ACTIVE;
             }
-            Slog.i(TAG, "Updated health check state for package " + getName() + ": "
-                    + toString(oldState) + " -> " + toString(mHealthCheckState));
+
+            if (oldState != mHealthCheckState) {
+                Slog.i(TAG, "Updated health check state for package " + getName() + ": "
+                        + toString(oldState) + " -> " + toString(mHealthCheckState));
+            }
             return mHealthCheckState;
         }
 
-- 
cgit v1.2.3-59-g8ed1b