Closed Bug 1462010 Opened 2 years ago Closed 2 years ago

Remove migrateRegionPrefs from search service

Categories

(Firefox :: Search, enhancement, P1)

enhancement

Tracking

()

RESOLVED FIXED
Firefox 62
Tracking Status
firefox62 --- fixed

People

(Reporter: mkaply, Assigned: mkaply)

References

Details

Attachments

(1 file)

We have a function migrateRegionPrefs that does migration based on the old isUS/countryCode prefs.

In Firefox 57, we did a one time reset of all these prefs (and we will do it for all old users that upgrade)

https://searchfox.org/mozilla-central/source/browser/components/nsBrowserGlue.js#2075

So there are no users that have these preferences anymore.

We should get rid of this unused code.

At the same time, we should get rid of all references to the old .isUS preference.

It's never set anymore and it is not used except in migration.
Blocks: 1462015
Comment on attachment 8976183 [details]
Bug 1462010 - Remove unused migrateRegionPrefs and isUS preference.

https://reviewboard.mozilla.org/r/244368/#review250566
Attachment #8976183 - Flags: review?(adw) → review+
We're sorry, Autoland could not rebase your commits for you automatically. Please manually rebase your commits and try again.

hg error in cmd: hg rebase -s 541b24452ba79cc2d863816bdca22fa895fa5e36 -d d53f1330f08c: rebasing 464090:541b24452ba7 "Bug 1462010 - Remove unused migrateRegionPrefs and isUS preference. r=adw" (tip)
merging testing/profiles/common/user.js
merging testing/profiles/reftest/user.js and layout/tools/reftest/reftest-preferences.js to testing/profiles/reftest/user.js
warning: conflicts while merging testing/profiles/common/user.js! (edit, then use 'hg resolve --mark')
warning: conflicts while merging testing/profiles/reftest/user.js! (edit, then use 'hg resolve --mark')
unresolved conflicts (see hg resolve, then hg rebase --continue)
We're sorry, Autoland could not rebase your commits for you automatically. Please manually rebase your commits and try again.

hg error in cmd: hg rebase -s 46fb193b8f55080b4191841e879d5a623fc35269 -d 18b348f11483: rebasing 464091:46fb193b8f55 "Bug 1462010 - Remove unused migrateRegionPrefs and isUS preference. r=adw" (tip)
merging testing/profiles/common/user.js
merging testing/profiles/reftest/user.js and layout/tools/reftest/reftest-preferences.js to testing/profiles/reftest/user.js
warning: conflicts while merging testing/profiles/common/user.js! (edit, then use 'hg resolve --mark')
warning: conflicts while merging testing/profiles/reftest/user.js! (edit, then use 'hg resolve --mark')
unresolved conflicts (see hg resolve, then hg rebase --continue)
Pushed by mozilla@kaply.com:
https://hg.mozilla.org/integration/autoland/rev/e993aa150201
Remove unused migrateRegionPrefs and isUS preference. r=adw
https://hg.mozilla.org/mozilla-central/rev/e993aa150201
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 62
You need to log in before you can comment on or make changes to this bug.