Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(661)

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/ChromeBackupWatcher.java

Issue 2508173002: Force first backup (Closed)
Patch Set: Created 4 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/android/java/src/org/chromium/chrome/browser/ChromeBackupWatcher.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/ChromeBackupWatcher.java b/chrome/android/java/src/org/chromium/chrome/browser/ChromeBackupWatcher.java
index 554b054c57e220a5432cff9127047d3ebf4d800e..e3477ceb7fcffc3c12dd863afa0f5ce65393117d 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/ChromeBackupWatcher.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/ChromeBackupWatcher.java
@@ -22,6 +22,8 @@ import org.chromium.components.signin.ChromeSigninController;
public class ChromeBackupWatcher {
private BackupManager mBackupManager;
+ private static final String FIRST_BACKUP_DONE = "first_backup_done";
+
@VisibleForTesting
@CalledByNative
static ChromeBackupWatcher createChromeBackupWatcher() {
@@ -35,6 +37,11 @@ public class ChromeBackupWatcher {
mBackupManager = new BackupManager(context);
// Watch the Java preferences that are backed up.
SharedPreferences sharedPrefs = ContextUtils.getAppSharedPreferences();
+ // If we have never done a backup do one immediately.
+ if (!sharedPrefs.getBoolean(FIRST_BACKUP_DONE, false)) {
+ mBackupManager.dataChanged();
+ sharedPrefs.edit().putBoolean(FIRST_BACKUP_DONE, true).apply();
Bernhard Bauer 2016/11/17 16:08:32 Actually, would it make sense to set this flag as
aberent 2016/11/17 16:58:54 First run on a new device will always change some
+ }
sharedPrefs.registerOnSharedPreferenceChangeListener(
new SharedPreferences.OnSharedPreferenceChangeListener() {
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698