1424343 - Stop migrating the browser.search.selectedEngine preference

Status: RESOLVED FIXED

(Firefox :: Search, defect)

Description

[Florian Quèze [:florian]]

We stopped using this preference in bug 1029148 for Firefox 34. In bug 1405670 we dropped migration code for search settings written before Firefox 45; we should have removed this pref migration code too.

Comment 1

[Florian Quèze [:florian]]

Attachment: Patch

Simple code removal. The only other reference to this pref in our code is the test at https://searchfox.org/mozilla-central/rev/f5f1c3f294f89cfd242c3af9eb2c40d19d5e04e7/toolkit/components/search/tests/xpcshell/test_selectedEngine.js#16 that actually verifies that the pref has no effect anymore.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/9a7bc6776fb7
Stop migrating the browser.search.selectedEngine preference, r=adw.

Comment 3

[Andrei Ciure[:aciure]]

https://hg.mozilla.org/mozilla-central/rev/9a7bc6776fb7

Comment 4

[Florian Quèze [:florian]]

Comment on attachment 8935861 [details] [diff] [review]
Patch

Approval Request Comment
[Feature/Bug causing the regression]: Not a regression, just something I missed in bug 1405670.
[User impact if declined]: Slight risk of abuse for an old preference to change the user's default search engine.
[Is this code covered by automated tests?]: no.
[Has the fix been verified in Nightly?]: no.
[Needs manual test from QE? If yes, steps to reproduce]: no.
[List of other uplifts needed for the feature/fix]: none
[Is the change risky?]: The risk is almost non-existant 
[Why is the change risky/not risky?]: Only removing some dead migration code that has stayed in the tree for far longer than initially expected.
[String changes made/needed]: none.

Comment 5

[Gerry Chang [:gchang]]

Comment on attachment 8935861 [details] [diff] [review]
Patch

Take this for follow-up of bug 1405670. Beta58+.

Comment 6

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

https://hg.mozilla.org/releases/mozilla-beta/rev/96355d26b932