Merge "Workaround: don't migrate grid when hotseat size or device type changes in multi display" into sc-v2-dev

This commit is contained in:
TreeHugger Robot
2021-09-20 15:19:58 +00:00
committed by Android (Google) Code Review
2 changed files with 16 additions and 4 deletions
@@ -24,6 +24,7 @@ import static com.android.launcher3.logging.StatsLogManager.LauncherEvent.LAUNCH
import android.content.Context;
import android.content.SharedPreferences;
import android.text.TextUtils;
import android.util.Log;
import androidx.annotation.IntDef;
@@ -134,8 +135,17 @@ public class DeviceGridState {
public boolean isCompatible(DeviceGridState other) {
if (this == other) return true;
if (other == null) return false;
return mNumHotseat == other.mNumHotseat
boolean isCompatible = mNumHotseat == other.mNumHotseat
&& deviceTypeCompatible(mDeviceType, other.mDeviceType)
&& Objects.equals(mGridSizeString, other.mGridSizeString);
// TODO(b/198965093): Temporary fix for multi-display devices, ignore hotseat size changes
// and type compatibility.
if ((mDeviceType == TYPE_MULTI_DISPLAY || other.mDeviceType == TYPE_MULTI_DISPLAY)
&& !isCompatible && Objects.equals(mGridSizeString, other.mGridSizeString)) {
Log.d("b/198965093", "Hotseat and deice type compatibility ignored: " + this
+ ", other: " + other);
isCompatible = true;
}
return isCompatible;
}
}
@@ -106,9 +106,11 @@ public class GridSizeMigrationTaskV2 {
DeviceGridState idpGridState = new DeviceGridState(idp);
DeviceGridState contextGridState = new DeviceGridState(context);
boolean needsToMigrate = !idpGridState.isCompatible(contextGridState);
// TODO: Revert this change after b/200010396 is fixed
Log.d(TAG, "Migration is needed. idpGridState: " + idpGridState
+ ", contextGridState: " + contextGridState);
// TODO(b/198965093): Revert this change after bug is fixed
if (needsToMigrate) {
Log.d("b/198965093", "Migration is needed. idpGridState: " + idpGridState
+ ", contextGridState: " + contextGridState);
}
return needsToMigrate;
}