957405 - Batch-process writing favicons to the database

Status: RESOLVED INCOMPLETE

(Firefox for Android Graveyard :: Favicon Handling, defect)

Description

[Chris Kitching [:ckitching]]

Currently, each time a favicon is loaded we make a query to the database to store it (In the case of multiple-image favicons, we still only make one query).

It may be possible to get a meaningful performance boost by doing this asynchronously. The saving of the favicon to the DB would no longer delay the loading of the favicon, and we get to eliminate query overheads.
This would probably take the form of some background thread that queues up "Save this favicon to the DB" requests and processes them all at once at a convenient moment.

Suggested by RNewman in Bug 748100 Comment 33.

Comment 1

[BMO Automation]

We have completed our launch of our new Firefox on Android. The development of the new versions use GitHub for issue tracking. If the bug report still reproduces in a current version of [Firefox on Android nightly](https://play.google.com/store/apps/details?id=org.mozilla.fenix) an issue can be reported at the [Fenix GitHub project](https://github.com/mozilla-mobile/fenix/). If you want to discuss your report please use [Mozilla's chat](https://wiki.mozilla.org/Matrix#Connect_to_Matrix) server https://chat.mozilla.org and join the [#fenix](https://chat.mozilla.org/#/room/#fenix:mozilla.org) channel.