Rename "managed (mode|user)" to "supervised user" (part 3)

chrome/browser/sync/backup_rollback_controller.h

Index: chrome/browser/sync/backup_rollback_controller.h
diff --git a/chrome/browser/sync/backup_rollback_controller.h b/chrome/browser/sync/backup_rollback_controller.h
index 5a3cd5e48f38683bb7f0f8203f5b5e2a5b584561..f4887a992bec183213b56ab87986cd85947e311b 100644
--- a/chrome/browser/sync/backup_rollback_controller.h
+++ b/chrome/browser/sync/backup_rollback_controller.h
@@ -8,7 +8,7 @@
 #include "base/callback.h"
 #include "base/memory/weak_ptr.h"
 
-class ManagedUserSigninManagerWrapper;
+class SupervisedUserSigninManagerWrapper;
 
 namespace sync_driver {
 class SyncPrefs;
@@ -23,7 +23,7 @@ namespace browser_sync {
 class BackupRollbackController {
  public:
   BackupRollbackController(sync_driver::SyncPrefs* sync_prefs,
-                           const ManagedUserSigninManagerWrapper* signin,
+                           const SupervisedUserSigninManagerWrapper* signin,
                            base::Closure start_backup,
                            base::Closure start_rollback);
   ~BackupRollbackController();
@@ -45,10 +45,10 @@ class BackupRollbackController {
 
   sync_driver::SyncPrefs* sync_prefs_;
 
-  // Use ManagedUserSigninManagerWrapper instead of SigninManagerBase because
-  // SigninManagerBase could return non-empty user name for managed user, which
-  // would cause backup to trumpet normal sync for managed user.
-  const ManagedUserSigninManagerWrapper* signin_;
+  // Use SupervisedUserSigninManagerWrapper instead of SigninManagerBase because
+  // SigninManagerBase could return non-empty user name for supervised user,
+  // which would cause backup to trumpet normal sync for supervised user.

[Pam (message me for reviews), 2014/06/20 20:56:48]

What does "backup to trumpet normal sync" mean?

[Marc Treib, 2014/06/23 16:31:30]

I'm guessing "trumpet" should read "trump", which would make a bit more sense.
From looking at the code, though, the issue seems to be that
SigninManager::GetAuthenticatedUsername *does* return an empty string for
supervised users.

[Pam (message me for reviews), 2014/06/23 17:00:24]

I'm not even sure what that means. What is "backup" here? Can we clarify the
comment (in another CL)?

On 2014/06/23 16:31:30, treib wrote:

+  const SupervisedUserSigninManagerWrapper* signin_;
 
   base::Closure start_backup_;
   base::Closure start_rollback_;