Add BackupRollbackController for starting backup/rollback

chrome/browser/sync/backup_rollback_controller.cc

+// Copyright (c) 2014 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "chrome/browser/sync/backup_rollback_controller.h"
+
+#include "base/command_line.h"
+#include "base/message_loop/message_loop.h"
+#include "chrome/browser/sync/managed_user_signin_manager_wrapper.h"
+#include "chrome/common/chrome_switches.h"
+#include "components/sync_driver/sync_prefs.h"
+
+namespace browser_sync {
+
+// Number of rollback attempts to try before giving up.
+static const int kRollbackLimits = 3;
+
+BackupRollbackController::BackupRollbackController(
+    sync_driver::SyncPrefs* sync_prefs,
+    const ManagedUserSigninManagerWrapper* signin,
+    base::Closure start_backup,
+    base::Closure start_rollback)
+    : sync_prefs_(sync_prefs),
+      signin_(signin),
+      start_backup_(start_backup),
+      start_rollback_(start_rollback),
+      weak_ptr_factory_(this) {
+}
+
+BackupRollbackController::~BackupRollbackController() {
+}
+
+void BackupRollbackController::Reset(base::TimeDelta delay) {
+#if defined(OS_WIN) || defined(OS_LINUX) || defined(OS_MACOSX)
+  if (!CommandLine::ForCurrentProcess()->HasSwitch(
+      switches::kSyncEnableBackupRollback)) {
+    return;
+  }
+
+  if (delay == base::TimeDelta()) {
+    TryStart();
+  } else {
+    base::MessageLoop::current()->PostDelayedTask(
+        FROM_HERE,
+        base::Bind(&BackupRollbackController::TryStart,
+                   weak_ptr_factory_.GetWeakPtr()),
+        delay);
+  }
+#endif
+}
+
+void BackupRollbackController::OnRollbackReceived() {
+#if defined(OS_WIN) || defined(OS_LINUX) || defined(OS_MACOSX)
+  sync_prefs_->SetRemainingRollbackTries(kRollbackLimits);

[Nicolas Zea, 2014/05/06 18:54:09]

Why do we have a limitted number of tries? What controls an attempt failing?

[haitaol1, 2014/05/07 21:34:17]

It's not good to be stuck in rollback mode. Backend is shut down on
unrecoverable error, which counts one rollback failure. If the unrecoverable
error is due to corrupted backup DB, setting limits on rollback will prevents
stuck.

On 2014/05/06 18:54:09, Nicolas Zea wrote:

[Nicolas Zea, 2014/05/08 21:43:09]

It seems odd that someone could prevent rollback by just restarting a couple
times. What's the consequence of being in a stuck rollback mode. That they can't
set up sync? That seems okay, since we stop attempting rollback once they sign
in, right?

[haitaol1, 2014/05/09 16:55:58]

The user changes not in backup db will be constantly removed. That would be a
bad user experience.


On 2014/05/08 21:43:09, Nicolas Zea wrote:

+#endif
+}
+
+void BackupRollbackController::OnRollbackDone() {
+#if defined(OS_WIN) || defined(OS_LINUX) || defined(OS_MACOSX)
+  sync_prefs_->SetRemainingRollbackTries(0);
+#endif
+}
+
+void BackupRollbackController::TryStart() {
+  if (!signin_->GetEffectiveUsername().empty()) {
+    DVLOG(1) << "Don't start backup/rollback when user is signed in.";
+    return;
+  }
+
+  int rollback_tries = sync_prefs_->GetRemainingRollbackTries();
+  if (rollback_tries > 0) {
+    DVLOG(1) << "Start rollback.";
+    sync_prefs_->SetRemainingRollbackTries(rollback_tries - 1);
+    start_rollback_.Run();
+  } else {
+    DVLOG(1) << "Start backup.";
+    start_backup_.Run();
+  }
+}
+
+}  // namespace browser_sync