Closed Bug 1170935 Opened 9 years ago Closed 7 years ago

Load home panel row favicons asynchronously

Categories

(Firefox for iOS :: Favicons, defect)

Product:
Firefox for iOS
Component:
Favicons
Platform:
All
iOS
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED WONTFIX

People

(Reporter: wesj, Unassigned)

Details

(Keywords: perf) 

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
Closed: 7 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.