summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Robert Carr <racarr@google.com> 2018-07-10 13:21:04 -0700
committer Robert Carr <racarr@google.com> 2018-07-10 13:21:26 -0700
commit6827f72fe2b03fae53c40c67beb830e8c811df11 (patch)
treedd3dc8ad7f87dcd1a2caf4ef43bbea795863135e
parent89c9915734601aaa405e8f7b9d51b7022ce999a0 (diff)
Make stop complete notification call in to ActivityManager blocking.
This call can lead directly to Surface destruction, so it's only safe to execute while the app renderer is paused. If the call is oneway and is deferred we can end up in a scenario where the call is processed (destroying the surface) after the client renderer has been unpaused for a different reason. Bug: 80423392 Bug: 62536731 Test: Manual. go/wm-smoke. Change-Id: Ide9da46a4ca7b6ef3b55c9966cc495b70b10fa46
-rw-r--r--core/java/android/app/IActivityTaskManager.aidl2
1 files changed, 1 insertions, 1 deletions
diff --git a/core/java/android/app/IActivityTaskManager.aidl b/core/java/android/app/IActivityTaskManager.aidl
index cba700345517..3dbc31250c01 100644
--- a/core/java/android/app/IActivityTaskManager.aidl
+++ b/core/java/android/app/IActivityTaskManager.aidl
@@ -128,7 +128,7 @@ interface IActivityTaskManager {
in boolean stopProfiling);
void activityResumed(in IBinder token);
void activityPaused(in IBinder token);
- oneway void activityStopped(in IBinder token, in Bundle state,
+ void activityStopped(in IBinder token, in Bundle state,
in PersistableBundle persistentState, in CharSequence description);
oneway void activityDestroyed(in IBinder token);
void activityRelaunched(in IBinder token);