Bug 1991953 - Set restored profile as default if old profile was default; rename to old-[profile name]. r=kpatenio

Differential Revision: https://phabricator.services.mozilla.com/D267664

Author: HarshitSohaney

browser/components/backup/BackupService.sys.mjs

@@ -2989,6 +2989,19 @@ export class BackupService extends EventTarget {
       );
       await IOUtils.writeJSON(postRecoveryPath, postRecovery);
 
+      // In a release scenario, this should always be true
+      // this makes it easier to get around setting up profiles for testing other functionality
+      if (profileSvc.currentProfile) {
+        // if our current profile was default, let's make the new one default
+        if (profileSvc.currentProfile === profileSvc.defaultProfile) {
+          profileSvc.defaultProfile = profile;
+        }
+
+        // let's rename the old profile with a prefix old-[profile_name]
+        profile.name = profileSvc.currentProfile.name;
+        profileSvc.currentProfile.name = `old-${profileSvc.currentProfile.name}`;
+      }
+
       await profileSvc.asyncFlush();
 
       if (shouldLaunch) {

browser/components/backup/tests/xpcshell/test_BackupService.js

@@ -273,6 +273,14 @@ async function testCreateBackupHelper(sandbox, taskFn) {
     "createBackupTestRecoveredProfile"
   );
 
+  let originalProfileName = currentProfile.name;
+
+  let profileSvc = Cc["@mozilla.org/toolkit/profile-service;1"].getService(
+    Ci.nsIToolkitProfileService
+  );
+  // make our current profile default
+  profileSvc.defaultProfile = currentProfile;
+
   let recoveredProfile = await bs.recoverFromBackupArchive(
     backupFilePath,
     null,
@@ -282,10 +290,22 @@ async function testCreateBackupHelper(sandbox, taskFn) {
   );
 
   Assert.ok(
-    recoveredProfile.name.startsWith(currentProfile.name),
+    recoveredProfile.name.startsWith(originalProfileName),
     "Should maintain profile name across backup and restore"
   );
 
+  Assert.strictEqual(
+    currentProfile.name,
+    `old-${originalProfileName}`,
+    "The old profile should be prefixed with old-"
+  );
+
+  Assert.strictEqual(
+    profileSvc.defaultProfile,
+    recoveredProfile,
+    "The new profile should now be the default"
+  );
+
   // Check that resources were recovered from highest to lowest backup priority.
   sinon.assert.callOrder(
     FakeBackupResource3.prototype.recover,