Closed Bug 811871 Opened 7 years ago Closed 4 years ago

Installing apps that have icons that use a // URL don't have icons showing up on install of the app

Categories

(Firefox OS Graveyard :: Gaia::Homescreen, defect)

ARM
Gonk (Firefox OS)
defect
Not set

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: marcia, Unassigned)

References

Details

(Keywords: unagi)

Attachments

(2 files)

Attached image Screenshot of issue
unagi, seen while running:

Gaia: 1c884f41292650615b04ca9f40cab981ea9e4d00
Gecko: 76848b5c67e6c164b28f366534e5eeab8fcadc2a

STR:
1. Install AirBnb and Helsa apps using the marketplace app
2. See attached screenshot
Desktop also reproduces this behavior. I think this a core bug that we don't recognize icon urls with the following style:

//a0.muscache.com/airbnb/static/mobile/site_icon_48-5a226c4c404103ee10a2ab988b45d3fa.png
Component: Gaia → DOM: Apps
Product: Boot2Gecko → Core
Version: unspecified → Trunk
Summary: [Apps] [homescreen] AirBnb and Helsa apps install without a favicon → Installing apps that have icons that use a // URL don't have icons showing up on install of the app
Fabrice - Should we even allow icon URLs like this?
We don't enforce anything on icons urls on the platform side, so gaia does whatever it can.

This is a valid relative uri though, that should resolve to http://a0... or https://a0... depending on the base URI used.
Component: DOM: Apps → Gaia
Product: Core → Boot2Gecko
Version: Trunk → unspecified
(In reply to Fabrice Desré [:fabrice] from comment #3)
> We don't enforce anything on icons urls on the platform side, so gaia does
> whatever it can.
> 
> This is a valid relative uri though, that should resolve to http://a0... or
> https://a0... depending on the base URI used.

Okay. From the triage perspective - I don't think I'd block ship on fixing this, unless you feel strongly that we should fix this.
Component: Gaia → Gaia::Homescreen
The issue still exists and is not only for the above mentioned apps. I downloaded few apps and they have issues too. Please see screenshot.

Verified  on Unagi Build ID: 20130225070200
Kernel: Dec 5
Gecko:  http://hg.mozilla.org/releases/mozilla-b2g18_v1_0_1/rev/3a5a27992a75
Gaia: 5691a16fff8e1403c75ed9d6f3a443b7e58198c6
Mass update: Resolve wontfix all issues with legacy homescreens.

As of 2.6 we have a new homescreen and having these issues open is confusing. All issues will block bug 1231115 so we can use that to re-visit any of these if needed.
Blocks: 1231115
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.