Update ProtoLogProxy classes to only log to protolog when protolog has been initialized

Flag: com.android.launcher3.enable_active_gesture_proto_log
Flag: com.android.launcher3.enable_recents_window_proto_log
Flag: com.android.launcher3.enable_state_manager_proto_log
Fixes: 381846204
Test: ran launcher and checked logs
Change-Id: I807326bd6c65b8e51f5302ba58eed841c23216f6
This commit is contained in:
Schneider Victor-Tulias
2024-12-12 11:03:56 -05:00
parent 74ff7af5e2
commit 7fb1d6d53b
4 changed files with 93 additions and 64 deletions

View File

@@ -18,6 +18,7 @@ package com.android.launcher3.util;
import static com.android.launcher3.Flags.enableStateManagerProtoLog;
import static com.android.quickstep.util.QuickstepProtoLogGroup.LAUNCHER_STATE_MANAGER;
import static com.android.quickstep.util.QuickstepProtoLogGroup.isProtoLogInitialized;
import androidx.annotation.NonNull;
@@ -30,7 +31,7 @@ public class StateManagerProtoLogProxy {
public static void logGoToState(
@NonNull Object fromState, @NonNull Object toState, @NonNull String trace) {
if (!enableStateManagerProtoLog()) return;
if (!enableStateManagerProtoLog() || !isProtoLogInitialized()) return;
ProtoLog.d(LAUNCHER_STATE_MANAGER,
"StateManager.goToState: fromState: %s, toState: %s, partial trace:\n%s",
fromState,
@@ -40,7 +41,7 @@ public class StateManagerProtoLogProxy {
public static void logCreateAtomicAnimation(
@NonNull Object fromState, @NonNull Object toState, @NonNull String trace) {
if (!enableStateManagerProtoLog()) return;
if (!enableStateManagerProtoLog() || !isProtoLogInitialized()) return;
ProtoLog.d(LAUNCHER_STATE_MANAGER, "StateManager.createAtomicAnimation: "
+ "fromState: %s, toState: %s, partial trace:\n%s",
fromState,
@@ -49,17 +50,17 @@ public class StateManagerProtoLogProxy {
}
public static void logOnStateTransitionStart(@NonNull Object state) {
if (!enableStateManagerProtoLog()) return;
if (!enableStateManagerProtoLog() || !isProtoLogInitialized()) return;
ProtoLog.d(LAUNCHER_STATE_MANAGER, "StateManager.onStateTransitionStart: state: %s", state);
}
public static void logOnStateTransitionEnd(@NonNull Object state) {
if (!enableStateManagerProtoLog()) return;
if (!enableStateManagerProtoLog() || !isProtoLogInitialized()) return;
ProtoLog.d(LAUNCHER_STATE_MANAGER, "StateManager.onStateTransitionEnd: state: %s", state);
}
public static void logCancelAnimation(boolean animationOngoing, @NonNull String trace) {
if (!enableStateManagerProtoLog()) return;
if (!enableStateManagerProtoLog() || !isProtoLogInitialized()) return;
ProtoLog.d(LAUNCHER_STATE_MANAGER,
"StateManager.cancelAnimation: animation ongoing: %b, partial trace:\n%s",
animationOngoing,