Closed
Bug 1642990
Opened 5 years ago
Closed 4 years ago
Ensure we are not writing/maintaining legacy search engine configuration data in the cache
Categories
(Firefox :: Search, task, P3)
Firefox
Search
Tracking
()
Tracking | Status | |
---|---|---|
firefox82 | --- | fixed |
People
(Reporter: standard8, Assigned: standard8)
References
Details
Attachments
(1 file)
(deleted),
text/x-phabricator-request
|
Details |
I suspect that once bug 1619926 is done, we shouldn't be writing any legacy configuration information to the cache.
However, we should have a quick check, and remove anything that we no longer need.
Comment hidden (off-topic) |
Assignee | ||
Updated•4 years ago
|
Type: defect → task
Flags: needinfo?(dharvey)
Updated•4 years ago
|
Severity: N/A → S3
Assignee | ||
Updated•4 years ago
|
Iteration: --- → 82.1 - Aug 24 - Sep 6
Assignee | ||
Updated•4 years ago
|
Assignee: nobody → standard8
Assignee | ||
Comment 2•4 years ago
|
||
appVersion is no longer used with modern config.
Also realised we could remove 'this._metaData' in the search service as it isn't used now.
Tidied up the load*FromCache functions as we don't need to pass the cache to them - just the engines. This makes the cache use clearer.
Also simplified getting the verified attribute when finding the default engine.
Depends on D88277
Pushed by mbanner@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/426d017d7e46
Stop writing appVersion to the search cache, clean up accesses. r=daleharvey
Pushed by mbanner@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/991fec7d58b3
Stop writing appVersion to the search cache, clean up accesses. r=daleharvey
Comment 5•4 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 4 years ago
status-firefox82:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 82 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•