commit e3a83064f9a86bffdcbce313e43ff4603627e2c1
author George Lin <giolin@google.com> Wed Feb 21 23:02:36 2024 +0000
committer George Lin <giolin@google.com> Thu Feb 22 14:48:38 2024 +0000
tree 9ffcd9991c7ae91807d2e5f8cb4b67d632f49390
parent e9333e7fa660254c1b12f5380f7d46e0efe8522e

Do not save crop hints to preferences (1/3)

Since we can get the currently set wallpaper's crop hints from the
wallpaper manager, there is no need to save the crop hints in the
prefereces, which may cause misalignment with the single source of
truth.

Saving information in the preferences is needed in 2 condition:
1. The information is not kept by the wallpaper manager
2. The information is needed for snapshot logging

Test: Manually tested it works as before
Bug: 309139122
Flag: ACONFIG com.android.wallpaper.multi_crop_preview_ui_flag DEVELOPMENT
Change-Id: Ia83764f55957f425448ec857a1e9db23176eada9

src/com/android/customization/module/ThemePickerInjector.kt

diff --git a/src/com/android/customization/module/ThemePickerInjector.kt b/src/com/android/customization/module/ThemePickerInjector.kt
index 8fd13d3..a2f7e56 100644
--- a/src/com/android/customization/module/ThemePickerInjector.kt
+++ b/src/com/android/customization/module/ThemePickerInjector.kt
@@ -214,7 +214,7 @@
                                 WallpaperClientImpl(
                                     context = appContext,
                                     wallpaperManager = WallpaperManager.getInstance(appContext),
-                                    wallpaperPreferences = getPreferences(appContext)
+                                    wallpaperPreferences = getPreferences(appContext),
                                 ),
                             wallpaperPreferences = getPreferences(context = appContext),
                             backgroundDispatcher = bgDispatcher,