Closed Bug 819895 Opened 12 years ago Closed 4 years ago

Different favicon for added search engine in urlbar/awesome bar search and about:addons (add-ons manager)

Categories

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

All
Android
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED INCOMPLETE

People

(Reporter: aryx, Unassigned)

References

Details

Attachments

(2 files)

Attached image screenshot url bar
Firefox for Android trunk 20121209, Android 4.1.2 (stock), Google Nexus S

The favicon for the search engine "Wikipedia SSL DE" I added to Firefox Mobile is different in the add-ons manager from the one shown in the urlbar. See attached screenshots.

Steps to reproduce:
1. Go to https://addons.mozilla.org/android/
2. Search for: Wikipedia SSL
3. Look for Wikipedia SSL (DE) and install it.
4. Type ggggg or something like this that won't be found in your history, notice the pixel garbage icon.
5. Open the add-ons manager from the Tools menu. Here you get what you expect.
Does a build prior to the patch landing in bug 808663 (Nov 15th) make any difference?
The addon provides a link to a 404 page for the icon:

https://secure.wikimedia.org/favicon.ico

Desktop shows the "Addon Manager" icon in this case. We could too.
I'm not sure how we're getting something "right" in the addons manager....
have you looked at the correct search engine? It's https://addons.mozilla.org/android/addon/wikipedia-ssl-de/?src=ss and the source looks like this:

<?xml version="1.0" encoding="UTF-8"?> 
<OpenSearchDescription xmlns="http://a9.com/-/spec/opensearch/1.1/" xmlns:moz="http://www.mozilla.org/2006/browser/search/"> 
  <ShortName>Wikipedia SSL DE</ShortName> 
  <Description>verschlüsselte Suche mit Wikipedia</Description> 
  <InputEncoding>UTF-8</InputEncoding> 
  <Image width="16" height="16" type="image/x-icon">data:image/x-icon;base64,AAABAAEAEBAQAAEABAAoAQAAFgAAACgAAAAQAAAAIAAAAAEABAAAAAAAAAAAAAAAAAAAAAAAEAAAAAAAAAAEAgQAhIOEAMjHyABIR0gA6ejpAGlqaQCpqKkAKCgoAPz9%2FAAZGBkAmJiYANjZ2ABXWFcAent6ALm6uQA8OjwAiIiIiIiIiIiIiI4oiL6IiIiIgzuIV4iIiIhndo53KIiIiB%2FWvXoYiIiIfEZfWBSIiIEGi%2FfoqoiIgzuL84i9iIjpGIoMiEHoiMkos3FojmiLlUipYliEWIF%2BiDe0GoRa7D6GPbjcu1yIiIiIiIiIiIiIiIiIiIiIiIiIiIiIiIgAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAA</Image> 
  <Url type="text/html" method="GET" template="https://de.wikipedia.org/wiki/Spezial:Suche">
    <Param name="search" value="{searchTerms}"/>
    <Param name="sourceid" value="Mozilla-search"/>
  </Url> 
  <Url type="application/x-suggestions+json" template="https://de.wikipedia.org/w/api.php">
    <Param name="action" value="opensearch"/>
    <Param name="search" value="{searchTerms}"/>
  </Url> 
  <moz:SearchForm>https://de.wikipedia.org/wiki/Spezial:Suche</moz:SearchForm> 
</OpenSearchDescription>

As you can see the image is embedded into the xml.
(In reply to Archaeopteryx [:aryx] from comment #5)
> have you looked at the correct search engine? It's

>   <Image width="16" height="16"
> type="image/x-icon">data:image/x-icon;base64,

> As you can see the image is embedded into the xml.

And it's 16px so we stretch it
The Favicon revamp of bug 888326 fixes this problem.
Assignee: nobody → ckitching
Depends on: 888326
Status: NEW → ASSIGNED
No longer depends on: 888326
Assignee: chriskitching → nobody
Component: General → Favicon Handling
Hardware: ARM → All
No assignee, updating the status.
Status: ASSIGNED → NEW
No assignee, updating the status.
No assignee, updating the status.
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.
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → INCOMPLETE
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: