Closed Bug 1076351 Opened 5 years ago Closed 5 years ago
Initialize the search activity default engine pref with the user's default search engine
I'm splitting this off of bug 1065891, since the patch I wrote there to try to address this caused an autophone regression. The main thing we want to do here is make sure when the user's browser is updated from a version that doesn't have the search activity to one that does (34 -> 35), the search activity default engine shared pref is set to the value of the user's default engine in Fennec (which they may have changed before the search activity logic was around to notice the change).
This is a simpler patch that doesn't try to add various migration logic. I added a check to make sure we only do this migration when the app is upgraded, so this won't affect new profiles. I also contemplated adding an observer to do this after the search service is initialized, but we're not guaranteed to start the search service while the app is running unless the user does something that requires it, and I would like to make sure this pref migration happens the first time Fennec is launched after updating.
Attachment #8499606 - Flags: review?(bnicholson)
Attachment #8499606 - Flags: review?(bnicholson) → review+
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 35
Flagging this for QE verification. Does this have any test coverage?
Verified as fixed on Firefox for Android 35 Beta 1 with Nexus 4 (Android 4.4.4).
You need to log in before you can comment on or make changes to this bug.