Open Bug 1029555 Opened 6 years ago Updated 2 years ago

No favicon for cnn search engine if added from Page Menu

Categories

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

ARM
Android
defect

Tracking

()

Tracking Status
firefox31 --- affected
firefox32 --- affected
firefox33 --- affected
firefox34 --- affected
firefox35 --- affected
firefox36 --- affected
firefox37 --- affected
firefox38 --- affected
firefox39 --- affected
firefox40 --- affected
fennec + ---

People

(Reporter: ioana.chiorean, Unassigned)

References

Details

Attachments

(1 file)

Attached image Screenshot
OS: Android 4.4.4, 4.1.2, 3.2.1
Devices: Alcatel One Touch, Nexus 4, Acer Iconia

Steps:
1. Go to cnn.com
2. Tap Menu -> Page -> Add a Search Engine (either one of the 2 will work )
3. Go back to cnn.com and long tap on the search fiedl (either one the page will work )
4. Go to Settings -> Customization -> Search 

Expected results:
- all search engines added should have a favicon

Actual Results:
- the one added from Page Menu has no favicon 
- log error: 
06-24 17:58:11.141: W/GeckoFaviconDecoder(5592): Unable to decode icon URI.

Note:
- reproducible on both phones and tablets
I don't think it is cnn specific problem only.
tracking-fennec: --- → ?
tracking-fennec: ? → +
Check to see if this behavior regressed.
Last good build: 2013-11-07;
First bad build: 2013-11-08;
Pushlog:
http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=70de5e24d79b&tochange=f003c386c77a
Ooh, that's useful. Thanks Flaviu!

The work done for Bug 852608 looks likely to be the culprit.

Also highly suspect is:
http://hg.mozilla.org/mozilla-central/rev/57da3b6e50c7#l1.43

Might it be that this wild guess is incorrect in the case of cnn?
filter on [mass-p5]
Priority: -- → P5
The reason this fails is we're only looking for {{ uri }}/favicon.ico. CNN's is located at <link href="http://www.cnn.com/favicon.ie9.ico" rel="Shortcut Icon" type="image/x-icon"/>.
Re-test this, they updated over new years and we get their new mobile site
Duplicate of this bug: 928585
Duplicate of this bug: 1170366
This is a problem on Firefox 50.0 in Ubuntu and 49.0.2 on Android. For example go to http://wiki.openstreetmap.org/ or https://www.wikidata.org and try adding it. They both have favicons, but they both appear as generic grey icons.
Also with Wiktionary.
Re-triaging per https://bugzilla.mozilla.org/show_bug.cgi?id=1473195

Needinfo :susheel if you think this bug should be re-triaged.
You need to log in before you can comment on or make changes to this bug.