[DataStore] Migrate LocaleNotification SharedPreferences to BackupRestoreStorage
Bug: 325144964
Test: Manual tests
(cherry picked from https://googleplex-android-review.googlesource.com/q/commit:cc0a94d73247fa81eff0054a0dcf06e99c45adb1)
Merged-In: I195ee91f29030fcfe7176ed0c18fb2e2d0d40257
Change-Id: I195ee91f29030fcfe7176ed0c18fb2e2d0d40257
diff --git a/src/com/android/settings/SettingsApplication.java b/src/com/android/settings/SettingsApplication.java
index 1a53b83..c0d2445 100644
--- a/src/com/android/settings/SettingsApplication.java
+++ b/src/com/android/settings/SettingsApplication.java
@@ -30,6 +30,7 @@
import com.android.settings.core.instrumentation.ElapsedTimeUtils;
import com.android.settings.fuelgauge.BatterySettingsStorage;
import com.android.settings.homepage.SettingsHomepageActivity;
+import com.android.settings.localepicker.LocaleNotificationDataManager;
import com.android.settings.overlay.FeatureFactory;
import com.android.settings.overlay.FeatureFactoryImpl;
import com.android.settings.spa.SettingsSpaEnvironment;
@@ -56,7 +57,10 @@
public void onCreate() {
super.onCreate();
- BackupRestoreStorageManager.getInstance(this).add(new BatterySettingsStorage(this));
+ BackupRestoreStorageManager.getInstance(this)
+ .add(
+ new BatterySettingsStorage(this),
+ LocaleNotificationDataManager.getSharedPreferencesStorage(this));
// Add null checking to avoid test case failed.
if (getApplicationContext() != null) {
diff --git a/src/com/android/settings/backup/SettingsBackupHelper.java b/src/com/android/settings/backup/SettingsBackupHelper.java
index 0861af2..556ab72 100644
--- a/src/com/android/settings/backup/SettingsBackupHelper.java
+++ b/src/com/android/settings/backup/SettingsBackupHelper.java
@@ -16,10 +16,8 @@
package com.android.settings.backup;
-import static com.android.settings.localepicker.LocaleNotificationDataManager.LOCALE_NOTIFICATION;
import android.app.backup.BackupAgentHelper;
-import android.app.backup.SharedPreferencesBackupHelper;
import com.android.settings.flags.Flags;
import com.android.settings.onboarding.OnboardingFeatureProvider;
@@ -29,15 +27,12 @@
/** Backup agent for Settings APK */
public class SettingsBackupHelper extends BackupAgentHelper {
- private static final String PREF_LOCALE_NOTIFICATION = "localeNotificationSharedPref";
public static final String SOUND_BACKUP_HELPER = "SoundSettingsBackup";
@Override
public void onCreate() {
super.onCreate();
BackupRestoreStorageManager.getInstance(this).addBackupAgentHelpers(this);
- addHelper(PREF_LOCALE_NOTIFICATION,
- new SharedPreferencesBackupHelper(this, LOCALE_NOTIFICATION));
if (Flags.enableSoundBackup()) {
OnboardingFeatureProvider onboardingFeatureProvider =
FeatureFactory.getFeatureFactory().getOnboardingFeatureProvider();
diff --git a/src/com/android/settings/localepicker/LocaleNotificationDataManager.java b/src/com/android/settings/localepicker/LocaleNotificationDataManager.java
index 0e89366..6ec578d 100644
--- a/src/com/android/settings/localepicker/LocaleNotificationDataManager.java
+++ b/src/com/android/settings/localepicker/LocaleNotificationDataManager.java
@@ -19,8 +19,11 @@
import android.content.Context;
import android.content.SharedPreferences;
+import androidx.annotation.NonNull;
import androidx.annotation.VisibleForTesting;
+import com.android.settingslib.datastore.SharedPreferencesStorage;
+
import com.google.gson.Gson;
import java.util.HashMap;
@@ -43,6 +46,12 @@
this.mContext = context;
}
+ /** Returns the underlying {@link SharedPreferences} storage. */
+ @NonNull
+ public static SharedPreferencesStorage getSharedPreferencesStorage(@NonNull Context context) {
+ return new SharedPreferencesStorage(context, LOCALE_NOTIFICATION, Context.MODE_PRIVATE);
+ }
+
private static SharedPreferences getSharedPreferences(Context context) {
return context.getSharedPreferences(LOCALE_NOTIFICATION, Context.MODE_PRIVATE);
}