Closed Bug 959896 Opened 9 years ago Closed 2 years ago

Home banner icon loaded from chrome:// URI is too small


(Firefox for Android Graveyard :: General, defect)

Not set


(Not tracked)



(Reporter: Margaret, Unassigned)



(Whiteboard: [qa+])


(1 file)

This sounds similar to bug 921668, but I'm experiencing this issue with chrome:// URIs now.

This loosely blocks bug 958891 because I'd like to include that icon as a chrome URI, but I can always fall back to just using a data URI if needed.

wesj, do you know where along the line this might be going wrong?
Flags: needinfo?(wjohnston)
I don't know what code path we're following, but if we're going down this path, we're not passing in the resources anywhere:
scaleType=center won't scale the image:

centerCrop,fitCenter, or centerInside will

Other than that, we should probably do some blog posts/write some utilities to help detect DPI and use it provide better/correct resolution resources from JS (as well as documenting the expected resolutions better).
Flags: needinfo?(wjohnston)
Whiteboard: [qa+]
But how does that explain the difference between chrome:// URIs and data: URIs? I tried loading the same image as a data: URI, and that actually made it large as expected.
I'm just going to use Android resources for bug 958891, so this doesn't block anymore, although it would still be good to investigate this bug at some point.
No longer blocks: 958891
Blocks: 1162107
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]( an issue can be reported at the [Fenix GitHub project]( If you want to discuss your report please use [Mozilla's chat]( server and join the [#fenix]( channel.
Closed: 2 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.