1244945 - Favicons get scaled on the UI thread

Status: RESOLVED INCOMPLETE

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

Description

[Michael Comella (:mcomella) [NI reported issues only: ex-Mozilla]]

LoadFaviconTask gets the favicon in doInBackground but then scales the bitmap in processResult which is called from onPostExecute.

Ideally, we should scale the bitmap in doInBackground and then call all the listeners on the UiThread. 

Realistically, this probably isn't too bad of a perf hit because most images will be coming from the cache already stored in memory so we only need to do it once per image but it can still be bad if we're processing many images.

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.