diff --git a/quickstep/src/com/android/quickstep/logging/StatsLogCompatManager.java b/quickstep/src/com/android/quickstep/logging/StatsLogCompatManager.java index b1c9ed0361..6575996d69 100644 --- a/quickstep/src/com/android/quickstep/logging/StatsLogCompatManager.java +++ b/quickstep/src/com/android/quickstep/logging/StatsLogCompatManager.java @@ -130,7 +130,8 @@ public class StatsLogCompatManager extends StatsLogManager { info.getAttribute().getNumber() /* origin */, getCardinality(info) /* cardinality */, info.getWidget().getSpanX(), - info.getWidget().getSpanY()); + info.getWidget().getSpanY(), + getFeatures(info)); } /** @@ -365,15 +366,12 @@ public class StatsLogCompatManager extends StatsLogManager { atomInfo.getFolderIcon().getFromLabelState().getNumber() /* fromState */, atomInfo.getFolderIcon().getToLabelState().getNumber() /* toState */, atomInfo.getFolderIcon().getLabelInfo() /* edittext */, - getCardinality(atomInfo) /* cardinality */); + getCardinality(atomInfo) /* cardinality */, + getFeatures(atomInfo) /* features */); } } private static int getCardinality(LauncherAtom.ItemInfo info) { - // TODO(b/187734511): Implement a unified solution for 1x1 widgets in folders/hotseat. - if (info.getItemCase().equals(LauncherAtom.ItemInfo.ItemCase.WIDGET)) { - return info.getWidget().getWidgetFeatures(); - } switch (info.getContainerInfo().getContainerCase()) { case PREDICTED_HOTSEAT_CONTAINER: return info.getContainerInfo().getPredictedHotseatContainer().getCardinality(); @@ -514,6 +512,13 @@ public class StatsLogCompatManager extends StatsLogManager { } } + private static int getFeatures(LauncherAtom.ItemInfo info) { + if (info.getItemCase().equals(LauncherAtom.ItemInfo.ItemCase.WIDGET)) { + return info.getWidget().getWidgetFeatures(); + } + return 0; + } + /** * Interface to get stats log while it is dispatched to the system