Closed
Bug 1621892
Opened 5 years ago
Closed 5 years ago
Search Engine settings are lost when a WebExtension based search engine is updated
Categories
(Firefox :: Search, defect, P2)
Firefox
Search
Tracking
()
People
(Reporter: standard8, Assigned: daleharvey)
References
Details
Attachments
(1 file)
(deleted),
text/x-phabricator-request
|
Details |
STR:
- Install an older version of a WebExtension with a search engine in it, e.g. I installed startpage 1.2.0 from here.
- Go to Preferences, Search, and then set it as default, change the order where it is, uncheck the tick box (hidden status) and add an alias.
- Restart so that settings are definitely saved (possibly don't need to do this step).
- Close the preferences tab.
- Go to about:addons and from the cog icon select "Check for Updates"
- Go back to the preferences tab.
Expected Results
All settings are saved.
Actual Results
The hidden status and alias have been lost. The engine does remain as default and in the same position in the list of engines.
Reporter | ||
Comment 1•5 years ago
|
||
The main issue here is that when a WebExtension is updated, the Add-on manager is first calling removeEngine
, and then calling addEnginesFromExtension
:
That code also explains why the position and default engines are kept.
Assignee | ||
Updated•5 years ago
|
Assignee: nobody → dharvey
Points: --- → 5
Assignee | ||
Updated•5 years ago
|
Iteration: --- → 76.1 - Mar 9 - Mar 22
Reporter | ||
Comment 2•5 years ago
|
||
We should probably add tests for:
- Updating from one version of a WebExtension to another, check that: all user settings retained, all urls/names etc updated.
- Updating from one version without a search engine to one with a search engine, check that: new engine is correctly added.
- Updating where the search engine is removed in the new WebExtension, check that: old engine is correctly removed.
I don't think the latter two will be happening often, but I think we should check for it anyway.
Assignee | ||
Comment 3•5 years ago
|
||
Updated•5 years ago
|
Iteration: 76.1 - Mar 9 - Mar 22 → 76.2 - Mar 23 - Apr 5
Assignee | ||
Updated•5 years ago
|
Iteration: 76.2 - Mar 23 - Apr 5 → 77.1 - Apr 6 - Apr 19
Pushed by dharvey@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/63b9f90d47b7
Maintain engine settings when upgrading extension r=Standard8,mixedpuppy
Comment 5•5 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 5 years ago
status-firefox77:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 77
Updated•4 years ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•