Load home panel row favicons asynchronously

RESOLVED WONTFIX

Status

()

RESOLVED WONTFIX
3 years ago
a year ago

People

(Reporter: wesj, Unassigned)

Tracking

({perf})

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

3 years ago
Doing some timing of our sql queries, the joins against the favicon database can approximately double their time. We might be better to run a quick query with no icons and then a second slower query with them? Or alternatively, to do what we do on Android and run an entirely separate favicons query for only ones that are visible?
Yeah, we should do async loading and avoid the join wherever possible.

In theory we can fetch the icons for only the first N in the main query itself, but it's probably not worth the effort.

(Also lol we have a favicons component)
Component: General → Favicons
Keywords: perf
Hardware: Other → All
Summary: Run home panel queries for favicons separately from basic queries → Load home panel row favicons asynchronously
Status: NEW → RESOLVED
Last Resolved: a year ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.