1090549 - Don't attempt to load favicons for pages that won't have them

Status: RESOLVED INCOMPLETE

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

Description

[Richard Newman [:rnewman]]

Spun off from Bug 1089940 Comment 17.

In short, if we're loading something like a data: URI page, we know that:

* The favicon is embedded in the 'page'
* We shouldn't write it to disk (the URI is bigger than the icon!)
* We should try reading it from disk
* We shouldn't try to guess a URI

In short: if we're a don't-try URI, just use LINK_FAVICON and check the memory cache.

See that bug for me.

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.