Closed
Bug 1937181
Opened 1 year ago
Closed 1 year ago
Allow passing the favicon URL into OpenSearchEngine to avoid setting it separately
Categories
(Firefox :: Search, task, P3)
Firefox
Search
Tracking
()
RESOLVED
FIXED
135 Branch
| Tracking | Status | |
|---|---|---|
| firefox135 | --- | fixed |
People
(Reporter: standard8, Assigned: standard8)
Details
Attachments
(1 file)
I noticed that when we're adding an OpenSearch engine, we are separately adding an icon URL after constructing the OpenSearchEngine object.
If we pass the icon URL into the constructor, then the OpenSearchEngine object can handle the icon and we can move more engine specific code out of SearchService.
| Assignee | ||
Comment 1•1 year ago
|
||
Pushed by mbanner@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/3e54190dbd92
Allow passing the favicon URL into OpenSearchEngine to avoid setting it separately. r=search-reviewers,scunnane
Status: NEW → RESOLVED
Closed: 1 year ago
status-firefox135:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 135 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•