From e2c4bcbf13bfc8aa62c91473d2327c33529032dc Mon Sep 17 00:00:00 2001 From: Jing Ji Date: Wed, 2 Jun 2021 14:00:53 -0700 Subject: [PATCH] Remove some system_server_wtf in proc stats tracking As those could be valid cases Bug: 18643865 Test: Manual Change-Id: I3391af28d8fe59e84b27a32889a6f4a4eaa42dea --- .../internal/app/procstats/AssociationState.java | 10 +++------- 1 file changed, 3 insertions(+), 7 deletions(-) diff --git a/core/java/com/android/internal/app/procstats/AssociationState.java b/core/java/com/android/internal/app/procstats/AssociationState.java index 86a356b47550f..97f4b0fc87330 100644 --- a/core/java/com/android/internal/app/procstats/AssociationState.java +++ b/core/java/com/android/internal/app/procstats/AssociationState.java @@ -103,12 +103,8 @@ public final class AssociationState { @Nullable private SourceState getCommonSourceState(boolean createIfNeeded) { - if (mCommonSourceState == null) { - if (createIfNeeded) { - mCommonSourceState = mTargetProcess.getOrCreateSourceState(mKey); - } else { - Slog.wtf(TAG, "Unable to find common source state for " + mKey.mProcess); - } + if (mCommonSourceState == null && createIfNeeded) { + mCommonSourceState = mTargetProcess.getOrCreateSourceState(mKey); } return mCommonSourceState; } @@ -225,7 +221,7 @@ public final class AssociationState { } mActiveProcState = mProcState; } - } else { + } else if (mAssociationState != null) { Slog.wtf(TAG, "startActive while not tracking: " + this); } if (mAssociationState != null) {