From 3da182844de8012004651d5fde94274bf70eb495 Mon Sep 17 00:00:00 2001 From: Federico Baron Date: Mon, 10 Mar 2025 18:45:25 +0000 Subject: Add logging for each migration type Flag: EXEMPT logging Test: n/a Bug: 393175684 Change-Id: Idfa551cb1e83c3c3bce498b6986035460798b93b --- src/com/android/launcher3/logging/StatsLogManager.java | 7 +++++++ src/com/android/launcher3/model/GridSizeMigrationDBController.java | 7 +++++++ src/com/android/launcher3/model/GridSizeMigrationLogic.kt | 7 +++++++ 3 files changed, 21 insertions(+) (limited to 'src') diff --git a/src/com/android/launcher3/logging/StatsLogManager.java b/src/com/android/launcher3/logging/StatsLogManager.java index 74d50988f1..45ed489dbe 100644 --- a/src/com/android/launcher3/logging/StatsLogManager.java +++ b/src/com/android/launcher3/logging/StatsLogManager.java @@ -880,6 +880,13 @@ public class StatsLogManager implements ResourceBasedOverride { @UiEvent(doc = "Work utility view shrink animation ended") LAUNCHER_WORK_UTILITY_VIEW_SHRINK_ANIMATION_END(2078), + + @UiEvent(doc = "Standard grid migration occurred") + LAUNCHER_STANDARD_GRID_MIGRATION(2200), + + @UiEvent(doc = "Row shift grid migration occurred") + LAUNCHER_ROW_SHIFT_GRID_MIGRATION(2201), + // ADD MORE ; diff --git a/src/com/android/launcher3/model/GridSizeMigrationDBController.java b/src/com/android/launcher3/model/GridSizeMigrationDBController.java index 5d0a7bd6a2..3e4394373a 100644 --- a/src/com/android/launcher3/model/GridSizeMigrationDBController.java +++ b/src/com/android/launcher3/model/GridSizeMigrationDBController.java @@ -20,6 +20,8 @@ import static com.android.launcher3.Flags.enableSmartspaceRemovalToggle; import static com.android.launcher3.LauncherSettings.Favorites.TABLE_NAME; import static com.android.launcher3.LauncherSettings.Favorites.TMP_TABLE; import static com.android.launcher3.Utilities.SHOULD_SHOW_FIRST_PAGE_WIDGET; +import static com.android.launcher3.logging.StatsLogManager.LauncherEvent.LAUNCHER_ROW_SHIFT_GRID_MIGRATION; +import static com.android.launcher3.logging.StatsLogManager.LauncherEvent.LAUNCHER_STANDARD_GRID_MIGRATION; import static com.android.launcher3.model.LoaderTask.SMARTSPACE_ON_HOME_SCREEN; import static com.android.launcher3.provider.LauncherDbUtils.copyTable; import static com.android.launcher3.provider.LauncherDbUtils.dropTable; @@ -44,6 +46,7 @@ import com.android.launcher3.LauncherPrefs; import com.android.launcher3.LauncherSettings; import com.android.launcher3.Utilities; import com.android.launcher3.config.FeatureFlags; +import com.android.launcher3.logging.StatsLogManager; import com.android.launcher3.provider.LauncherDbUtils.SQLiteTransaction; import com.android.launcher3.util.GridOccupancy; import com.android.launcher3.util.IntArray; @@ -127,6 +130,8 @@ public class GridSizeMigrationDBController { return true; } + StatsLogManager statsLogManager = StatsLogManager.newInstance(context); + boolean shouldMigrateToStrictlyTallerGrid = (Flags.oneGridSpecs() || isDestNewDb) && srcDeviceState.getColumns().equals(destDeviceState.getColumns()) && srcDeviceState.getRows() < destDeviceState.getRows(); @@ -152,6 +157,7 @@ public class GridSizeMigrationDBController { // Save current configuration, so that the migration does not run again. destDeviceState.writeToPrefs(context); t.commit(); + statsLogManager.logger().log(LAUNCHER_ROW_SHIFT_GRID_MIGRATION); return true; } @@ -163,6 +169,7 @@ public class GridSizeMigrationDBController { destDeviceState.getNumHotseat(), targetSize, srcDeviceState, destDeviceState); dropTable(t.getDb(), TMP_TABLE); t.commit(); + statsLogManager.logger().log(LAUNCHER_STANDARD_GRID_MIGRATION); return true; } catch (Exception e) { Log.e(TAG, "Error during grid migration", e); diff --git a/src/com/android/launcher3/model/GridSizeMigrationLogic.kt b/src/com/android/launcher3/model/GridSizeMigrationLogic.kt index 5df135a98f..2957e3c608 100644 --- a/src/com/android/launcher3/model/GridSizeMigrationLogic.kt +++ b/src/com/android/launcher3/model/GridSizeMigrationLogic.kt @@ -30,6 +30,9 @@ import com.android.launcher3.LauncherSettings.Favorites.TMP_TABLE import com.android.launcher3.Utilities import com.android.launcher3.config.FeatureFlags import com.android.launcher3.logging.FileLog +import com.android.launcher3.logging.StatsLogManager +import com.android.launcher3.logging.StatsLogManager.LauncherEvent.LAUNCHER_ROW_SHIFT_GRID_MIGRATION +import com.android.launcher3.logging.StatsLogManager.LauncherEvent.LAUNCHER_STANDARD_GRID_MIGRATION import com.android.launcher3.model.GridSizeMigrationDBController.DbReader import com.android.launcher3.provider.LauncherDbUtils.SQLiteTransaction import com.android.launcher3.provider.LauncherDbUtils.copyTable @@ -58,6 +61,8 @@ class GridSizeMigrationLogic { return } + val statsLogManager: StatsLogManager = StatsLogManager.newInstance(context) + val isAfterRestore = get(context).get(LauncherPrefs.IS_FIRST_LOAD_AFTER_RESTORE) FileLog.d( TAG, @@ -90,6 +95,7 @@ class GridSizeMigrationLogic { // Save current configuration, so that the migration does not run again. destDeviceState.writeToPrefs(context) t.commit() + statsLogManager.logger().log(LAUNCHER_ROW_SHIFT_GRID_MIGRATION) return } @@ -119,6 +125,7 @@ class GridSizeMigrationLogic { dropTable(t.db, TMP_TABLE) t.commit() + statsLogManager.logger().log(LAUNCHER_STANDARD_GRID_MIGRATION) } } catch (e: Exception) { FileLog.e(TAG, "Error during grid migration", e) -- cgit v1.2.3-59-g8ed1b