Bug 1746804 - When migrating browser.ctrlTab.recentlyUsedOrder, stop clearing it so there's a backup in case of wrongful migration. r=mconley
authorDão Gottwald <dao@mozilla.com>
Thu, 06 Jan 2022 20:51:31 +0000
changeset 603920 0d09faf561bb4115178ca1985a6aef74f1ef31cf
parent 603919 2c72c4b8fb280bc23ae603f3d062f3987cbf9803
child 603921 7fc30867f03da567de17c3db30a091e7bdbdda2e
push id39124
push usersmolnar@mozilla.com
push dateFri, 07 Jan 2022 03:49:10 +0000
treeherdermozilla-central@611ec542121f [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersmconley
bugs1746804
milestone97.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1746804 - When migrating browser.ctrlTab.recentlyUsedOrder, stop clearing it so there's a backup in case of wrongful migration. r=mconley Differential Revision: https://phabricator.services.mozilla.com/D135206
browser/components/BrowserGlue.jsm
--- a/browser/components/BrowserGlue.jsm
+++ b/browser/components/BrowserGlue.jsm
@@ -3939,17 +3939,16 @@ BrowserGlue.prototype = {
           newPrefValue
         );
       } else {
         Services.prefs.setBoolPref(
           "browser.ctrlTab.sortByRecentlyUsed",
           defaultValue
         );
       }
-      Services.prefs.clearUserPref(oldPrefName);
     }
 
     if (currentUIVersion < 109) {
       // Migrate old pref to new pref
       if (
         Services.prefs.prefHasUserValue("signon.recipes.remoteRecipesEnabled")
       ) {
         // Fetch the previous value of signon.recipes.remoteRecipesEnabled and assign it to signon.recipes.remoteRecipes.enabled.