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)

task

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.

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
Resolution: --- → FIXED
Target Milestone: --- → 135 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: