Favicon missing on URL bar; visible in awesome screen

VERIFIED FIXED

Status

()

Firefox for Android
General
P3
normal
VERIFIED FIXED
6 years ago
6 years ago

People

(Reporter: aaronmt, Assigned: mfinkle)

Tracking

unspecified
ARM
Android
Points:
---

Firefox Tracking Flags

(firefox11 fixed, fennec11+)

Details

Attachments

(1 attachment)

(Reporter)

Description

6 years ago
Using the website: http://www.torontoist.com *, once the page has completed loading I do not see the site's associated Favicon. The browser will display the default image. However, the site's Favicon is displayed appropriately in their entries in the AwesomeScreen.

* <link rel="shortcut icon" href="http://torontoist.com/wp-content/themes/torontoist_theme20112/images/graphics/favicon.ico">

Not all sites do this, the issue seems spotty.

--
Samsung Nexus S (Android 2.3.6)
20111123040207
http://hg.mozilla.org/projects/birch/rev/cd5725c23a13
(Assignee)

Updated

6 years ago
Assignee: nobody → lucasr.at.mozilla
Priority: -- → P3
Created attachment 579957 [details] [diff] [review]
patch

Fixes a bug where the "target" was being treated like a HTMLDocument, but it's an HTMLLinkElement. Also fixed a 'this' binding problem in a different part of the code.
Assignee: lucasr.at.mozilla → mark.finkle
Attachment #579957 - Flags: review?(mbrubeck)
Attachment #579957 - Flags: review?(mbrubeck) → review+
https://hg.mozilla.org/integration/mozilla-inbound/rev/ed8c9124bef5
(Reporter)

Comment 3

6 years ago
Samsung Galaxy SII (Android 2.3.4)
20111208060054
http://hg.mozilla.org/integration/mozilla-inbound/rev/b0f8871174a5
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
(Reporter)

Updated

6 years ago
Status: RESOLVED → VERIFIED
(In reply to Aaron Train [:aaronmt] from comment #3)
> Samsung Galaxy SII (Android 2.3.4)
> 20111208060054
> http://hg.mozilla.org/integration/mozilla-inbound/rev/b0f8871174a5

Mis-fire. Wrong cset and code is not on m-c yet.
Status: VERIFIED → REOPENED
Resolution: FIXED → ---
(Assignee)

Updated

6 years ago
Status: REOPENED → NEW

Comment 5

6 years ago
https://hg.mozilla.org/mozilla-central/rev/ed8c9124bef5
Status: NEW → RESOLVED
Last Resolved: 6 years ago6 years ago
Resolution: --- → FIXED

Comment 6

6 years ago
Build ID: Mozilla/5.0 (Android; Linux armv7l; rv:11.0a1) Gecko/20111211 Firefox/11.0a1 Fennec/11.0a1
Device: Samsung Nexus S
OS: Android 2.3

Favicon displayed on URL bar and awesome screen.(Verified on http://www.torontoist.com, bugzilla.mozilla.org, cnn.com, google.com, getfirebug.com, ... )

Marking as verified.
Status: RESOLVED → VERIFIED
tracking-fennec: --- → 11+
status-firefox11: --- → fixed
You need to log in before you can comment on or make changes to this bug.