1908555 - Close the current profile after launching a new one when restoring a backup

Status: RESOLVED FIXED

(Firefox :: Profile Backup, task, P1)

Description

[Fred Chasen [:fchasen]]

Currently after restoring from a backup, the new profile instance is launched behind the current window.
We should close the existing profile's instance after the new one has launched.

Comment 1

[Fred Chasen [:fchasen]]

While quitting the current instance worked, the experience is a little jarring as the new profile isn't focused / in the foreground after launch, so it was hidden behind another window for me. While nsAppStartup::CreateInstanceWithProfile seems to have a reference to the process it created but no idea if that is enough to focus it.

Seem that a better approach will be to just set ProfileService.defaultProfile to the new profile and then restart.

Will add an option to the recovery method to restart to do this.

Comment 2

[Fred Chasen [:fchasen]]

Attachment: WIP: Bug 1908555 - Pass foreground argument to new instance when using CreateInstanceWithProfile on a Mac.

Comment 3

[Fred Chasen [:fchasen]]

Attachment: Bug 1908555 - Set recovered profile to default and quit current instance after restoring from backup. r?#backup-reviewers!,mossop!

• Adds a switchToProfile method to the Backup Service which will set a profile to default and create a new instance.
• Adds a call to quit the current instance after a backup recovery has finished successfully.
• Adds browser.backup.recovery.quit-after-recovery and browser.backup.recovery.set-recovered-profile-as-default prefs to disable the above for testing.

Comment 4

[Harshit Sohaney [:hsohaney]]

Fixed in Bug 1993087