summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Nadav Bar <nadavbar@google.com> 2020-08-06 12:25:19 +0300
committer Nadav Bar <nadavbar@google.com> 2020-08-11 09:44:05 +0300
commit762e0f9a2e9945a8b8cf1c3c93e3b70af8201908 (patch)
tree5ee788d844973692e485d530c61218a8f5beb582
parent9547241ba12a2b89609484217ec8ea7d276c7601 (diff)
Fix SystemCaptionsManagerService to re-bind after a force stop
This change also fixes a bug in AbstractMasterSystemService where following a "force stop", the per-user service was not updated if PACKAGE_RESTART_POLICY_REFRESH_EAGER was specified. These behavior resulted with getServiceComponentName returning null and the service not being restarted. Bug: 160735520 Bug: 160693548 Bug: 160644088 Change-Id: I2f66273bd90e9bb220f799d8ab3b024637b33c24 Test: Manually
-rw-r--r--services/core/java/com/android/server/infra/AbstractMasterSystemService.java2
-rw-r--r--services/systemcaptions/java/com/android/server/systemcaptions/SystemCaptionsManagerService.java3
2 files changed, 3 insertions, 2 deletions
diff --git a/services/core/java/com/android/server/infra/AbstractMasterSystemService.java b/services/core/java/com/android/server/infra/AbstractMasterSystemService.java
index 7bbcdaa2d473..6672daa6f17a 100644
--- a/services/core/java/com/android/server/infra/AbstractMasterSystemService.java
+++ b/services/core/java/com/android/server/infra/AbstractMasterSystemService.java
@@ -949,7 +949,7 @@ public abstract class AbstractMasterSystemService<M extends AbstractMasterSystem
if (debug) {
Slog.d(mTag, "Eagerly recreating service for user " + userId);
}
- getServiceForUserLocked(userId);
+ updateCachedServiceLocked(userId);
}
}
onServicePackageRestartedLocked(userId);
diff --git a/services/systemcaptions/java/com/android/server/systemcaptions/SystemCaptionsManagerService.java b/services/systemcaptions/java/com/android/server/systemcaptions/SystemCaptionsManagerService.java
index 27a116c8043e..1586a33ba0e9 100644
--- a/services/systemcaptions/java/com/android/server/systemcaptions/SystemCaptionsManagerService.java
+++ b/services/systemcaptions/java/com/android/server/systemcaptions/SystemCaptionsManagerService.java
@@ -34,7 +34,8 @@ public final class SystemCaptionsManagerService extends
context,
com.android.internal.R.string.config_defaultSystemCaptionsManagerService),
/*disallowProperty=*/ null,
- /*packageUpdatePolicy=*/ PACKAGE_UPDATE_POLICY_REFRESH_EAGER);
+ /*packageUpdatePolicy=*/ PACKAGE_UPDATE_POLICY_REFRESH_EAGER
+ | PACKAGE_RESTART_POLICY_REFRESH_EAGER);
}
@Override