diff options
| author | 2018-07-10 13:21:04 -0700 | |
|---|---|---|
| committer | 2018-07-10 13:21:26 -0700 | |
| commit | 6827f72fe2b03fae53c40c67beb830e8c811df11 (patch) | |
| tree | dd3dc8ad7f87dcd1a2caf4ef43bbea795863135e | |
| parent | 89c9915734601aaa405e8f7b9d51b7022ce999a0 (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.aidl | 2 |
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); |