From c77b3822a79001abe656b91bf1d361870fefd99e Mon Sep 17 00:00:00 2001
From: Sharon Su <susharon@google.com>
Date: Fri, 28 Apr 2023 00:59:18 +0000
Subject: [PATCH] Change timeout idle settings from permanent binding to 2
 minutes. Because the cinematic wallpaper feature is not a frequently used
 feature, the service side implementation can choose to use a non-persistent
 process. Change from permanent binding to timeout on idle helps to kill the
 non-persistent process if service implementation picks up that option.

Test: manual and cts
Bug: 279988914
Change-Id: I0bdc855cbefb98df896372a8ea91e56d340c64b6
---
 .../RemoteWallpaperEffectsGenerationService.java              | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/services/wallpapereffectsgeneration/java/com/android/server/wallpapereffectsgeneration/RemoteWallpaperEffectsGenerationService.java b/services/wallpapereffectsgeneration/java/com/android/server/wallpapereffectsgeneration/RemoteWallpaperEffectsGenerationService.java
index c228dafcee8e0..61f34c2570ff4 100644
--- a/services/wallpapereffectsgeneration/java/com/android/server/wallpapereffectsgeneration/RemoteWallpaperEffectsGenerationService.java
+++ b/services/wallpapereffectsgeneration/java/com/android/server/wallpapereffectsgeneration/RemoteWallpaperEffectsGenerationService.java
@@ -40,6 +40,8 @@ public class RemoteWallpaperEffectsGenerationService extends
 
     private static final long TIMEOUT_REMOTE_REQUEST_MILLIS = 2 * DateUtils.SECOND_IN_MILLIS;
 
+    private static final long TIMEOUT_IDLE_BIND_MILLIS = 120 * DateUtils.SECOND_IN_MILLIS;
+
     private final RemoteWallpaperEffectsGenerationServiceCallback mCallback;
 
     public RemoteWallpaperEffectsGenerationService(Context context,
@@ -62,7 +64,7 @@ public class RemoteWallpaperEffectsGenerationService extends
 
     @Override
     protected long getTimeoutIdleBindMillis() {
-        return PERMANENT_BOUND_TIMEOUT_MS;
+        return TIMEOUT_IDLE_BIND_MILLIS;
     }
 
     @Override