From 6dddb3b9c98d353edaa3bd900ea262c8b1e0ee12 Mon Sep 17 00:00:00 2001 From: davotoula Date: Fri, 17 Jul 2026 16:51:28 +0100 Subject: [PATCH] Code review: - clarify no-op lifecycle callback comments --- .../amethyst/service/resourceusage/ForegroundTracker.kt | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/amethyst/src/main/java/com/vitorpamplona/amethyst/service/resourceusage/ForegroundTracker.kt b/amethyst/src/main/java/com/vitorpamplona/amethyst/service/resourceusage/ForegroundTracker.kt index 98c52d21ea..0904ed1c3a 100644 --- a/amethyst/src/main/java/com/vitorpamplona/amethyst/service/resourceusage/ForegroundTracker.kt +++ b/amethyst/src/main/java/com/vitorpamplona/amethyst/service/resourceusage/ForegroundTracker.kt @@ -61,11 +61,11 @@ class ForegroundTracker : Application.ActivityLifecycleCallbacks { } override fun onActivityResumed(activity: Activity) { - // No-op: resume is not observable and stays false during PiP / dialogs. + // No-op: foreground is derived from started/stopped counts, not resume. } override fun onActivityPaused(activity: Activity) { - // No-op: pause is not observable and stays false during PiP / dialogs. + // No-op: foreground is derived from started/stopped counts, not pause. } override fun onActivitySaveInstanceState(