Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "Fix race condition on size compat UI" into sc-qpr1-dev am: f9c3101df3

Browse Source 
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/15614097

Change-Id: I6f4dec56c0bccbbda0d2a58d18cb20190ab29917
This commit is contained in:
Chris Li
2021-08-18 23:47:53 +00:00
committed by Automerger Merge Worker
parent 242c8ea4bc f9c3101df3
commit a60a0a7cad
2 changed files with 8 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
libs/WindowManager/Shell/src/com/android/wm/shell/sizecompatui/SizeCompatUILayout.java
View File

@@ -28,6 +28,7 @@ import android.content.res.Configuration;
import android.graphics.PixelFormat;
import android.graphics.Rect;
import android.os.Binder;
import android.util.Log;
import android.view.SurfaceControl;
import android.view.View;
import android.view.WindowManager;
@@ -45,7 +46,7 @@ import com.android.wm.shell.common.SyncTransactionQueue;
class SizeCompatUILayout {
private static final String TAG = "SizeCompatUILayout";
private final SyncTransactionQueue mSyncQueue;
final SyncTransactionQueue mSyncQueue;
private final SizeCompatUIController.SizeCompatUICallback mCallback;
private Context mContext;
private Configuration mTaskConfig;
@@ -306,6 +307,10 @@ class SizeCompatUILayout {
private void updateSurfacePosition(SurfaceControl leash, int positionX, int positionY) {
mSyncQueue.runInSync(t -> {
if (!leash.isValid()) {
Log.w(TAG, "The leash has been released.");
return;
}
t.setPosition(leash, positionX, positionY);
// The size compat UI should be the topmost child of the Task in case there can be more
// than one children.

3
libs/WindowManager/Shell/src/com/android/wm/shell/sizecompatui/SizeCompatUIWindowManager.java
View File

@@ -110,7 +110,8 @@ class SizeCompatUIWindowManager extends WindowlessWindowManager {
}
if (mLeash != null) {
new SurfaceControl.Transaction().remove(mLeash).apply();
final SurfaceControl leash = mLeash;
mLayout.mSyncQueue.runInSync(t -> t.remove(leash));
mLeash = null;
}
}