[Sync] Fix sync backup for supervised users

chrome/browser/sync/profile_sync_service.cc

Index: chrome/browser/sync/profile_sync_service.cc
diff --git a/chrome/browser/sync/profile_sync_service.cc b/chrome/browser/sync/profile_sync_service.cc
index a23f96e15951b9072bede72ef2dc340e608df217..ffcbbc461ad43d68a260237aa745a21234d19168 100644
--- a/chrome/browser/sync/profile_sync_service.cc
+++ b/chrome/browser/sync/profile_sync_service.cc
@@ -328,8 +328,11 @@ void ProfileSyncService::Initialize() {
   if (browser_sync::BackupRollbackController::IsBackupEnabled()) {
     // Backup is needed if user's not signed in or signed in but previous
     // backup didn't finish, i.e. backend didn't switch from backup to sync.
-    need_backup_ = signin_->GetEffectiveUsername().empty() ||
-        sync_prefs_.GetFirstSyncTime().is_null();
+    // Note that backup is never performed for supervised users (which always
+    // start with a fresh profile anyways).
+    need_backup_ = (signin_->GetEffectiveUsername().empty() ||

[Marc Treib, 2014/09/25 09:31:51]

If you want to exclude supervised users, I think it would be easier to not use
the SigninManagerWrapper here, and instead just ask the SigninManager directly
for the username, i.e.
signin_->GetOriginal()->GetAuthenticatedUsername().empty()
That way you don't need the extra IsSupervised check.

[Nicolas Zea, 2014/09/26 01:26:48]

Unfortunately I don't think that will work. The first sync time is still null. I
need to distinguish that its a supervised user and have that override the other
logic.

That said, the alternative is to fix the logic in supervised user service and
make backup work properly. The new patchset does that (although the integration
tests need more work to have proper coverage. Filed 417930 for that).

+        sync_prefs_.GetFirstSyncTime().is_null()) &&
+        !profile_->IsSupervised();
 
     // Try to resume rollback if it didn't finish in last session.
     running_rollback = backup_rollback_controller_.StartRollback();