Closed Bug 1882577 Opened 4 months ago Closed 3 months ago

Make nsISearchEngine.getIconURL an async function

Categories

(Firefox :: Search, task, P3)

task

Tracking

()

RESOLVED FIXED
125 Branch
Tracking Status
firefox125 --- fixed

People

(Reporter: standard8, Assigned: standard8)

References

Details

Attachments

(1 file)

In bug 1867288 we are changing the application provided engines (defined by the new search configuration) to load icons from remote settings.

This means we won't necessarily be storing icons in memory in the engines themselves, and we'll need to load them from the remote settings store (which has local dumps/copies of the files), hence the ability to load them asynchronously is needed.

Hence this first bug is for changing nsISearchEngine.getIconURL to an async function. This is ready to go, so we can get it landed and being tested whilst we finish off the the main patch.

Attachment #9388182 - Attachment description: Bug 1882577 - Make nsISearchEngine.getIconURL an async function. r?#search-reviewers! → WIP: Bug 1882577 - Make nsISearchEngine.getIconURL an async function. r?#search-reviewers!
Depends on: 1884181
Attachment #9388182 - Attachment description: WIP: Bug 1882577 - Make nsISearchEngine.getIconURL an async function. r?#search-reviewers! → Bug 1882577 - Make nsISearchEngine.getIconURL an async function. r?#search-reviewers!
Pushed by mbanner@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/0dd16da50a5d
Make nsISearchEngine.getIconURL an async function. r=search-reviewers,extension-reviewers,settings-reviewers,robwu,mcheang,mak
Status: NEW → RESOLVED
Closed: 3 months ago
Resolution: --- → FIXED
Target Milestone: --- → 125 Branch
See Also: → 1885274
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: