701834 - Missing favicons for some sites

Status: VERIFIED FIXED

(Firefox for Android Graveyard :: General, defect, P2)

Description

[Dave Hunt [:davehunt] [he/him] ⌚GMT]

I noticed that a few sites aren't displaying favicons. The common factor is that the rel attribute for the <link> is 'shortcut icon' rather than just 'icon'.

Steps to reproduce:
1. Load getfirebug.com or docs.python.org

Expected result:
Favicon is displayed in the awesomebar

Actual result:
No favicon is displayed in the awesomebar

Comment 1

[John Hammink]

Confirmed on Android 3.1  (Samsung Galaxy Tab 10.1);
and Android 3.2  (Asus Transformer)

confirmed one reopened bug in the process:
https://bugzilla.mozilla.org/show_bug.cgi?id=697773

Comment 2

[Brian Nicholson (:bnicholson)]

Attachment: patch

Looks like there was a typo in Tab.java.

Comment 3

[Brian Nicholson (:bnicholson)]

http://hg.mozilla.org/projects/birch/rev/a81f3461217f

Comment 4

[Catalin Suciu [:csuciu]]

I verified this and noticed that the bug is fixed for getfirebug.com and docs.python.org but for cnn.com the favicon is still missing. 
Is this related to this bug? If not, please close this one and I will file a new bug.

Comment 5

[Brian Nicholson (:bnicholson)]

I can't reproduce this for cnn.com.  It's also possible that your problem is related to bug 701839.  If neither of these fix it, go ahead and file a new bug.

Comment 6

[Catalin Suciu [:csuciu]]

I cannot reproduce the cnn.com favicon issue on the latest fennec native build:
Mozilla/5.0 (Android;Linux armv7l;rv:11.0a1)Gecko/20111120 Firefox/11.0a1 Fennec/11.0a1

Closing.