Ignore the negative minWidth/minHeight populated from windowLayout

When an activity specifies defaultWidth/defaultHeight in <layout>
without minWidth/minHeight, -1 will be populated into windowLayout for
those fields.

Bug: 152933995
Test: manual enter/exit PiP from Duo app
Change-Id: I9056891f2baafbad9268a45a6820ec515d6f1b84
This commit is contained in:
Hongwei Wang
2020-04-02 17:13:08 -07:00
parent 2328849cc6
commit 907fd630a6

View File

@@ -37,7 +37,6 @@ import android.graphics.Rect;
import android.os.Handler;
import android.os.IBinder;
import android.os.Looper;
import android.os.RemoteException;
import android.util.Log;
import android.util.Size;
import android.view.SurfaceControl;
@@ -541,7 +540,12 @@ public class PipTaskOrganizer extends TaskOrganizer {
return null;
}
final ActivityInfo.WindowLayout windowLayout = activityInfo.windowLayout;
return new Size(windowLayout.minWidth, windowLayout.minHeight);
// -1 will be populated if an activity specifies defaultWidth/defaultHeight in <layout>
// without minWidth/minHeight
if (windowLayout.minWidth > 0 && windowLayout.minHeight > 0) {
return new Size(windowLayout.minWidth, windowLayout.minHeight);
}
return null;
}
private float getAspectRatioOrDefault(@Nullable PictureInPictureParams params) {