Closed Bug 1114998 Opened 10 years ago Closed 4 years ago

[meta] Significant resource space regression between 33 and 37

Categories

(Firefox for Android Graveyard :: Theme and Visual Design, defect)

37 Branch
All
Android
defect
Not set
normal

Tracking

(firefox37 affected, fennec-)

RESOLVED INCOMPLETE
Tracking Status
firefox37 --- affected
fennec - ---

People

(Reporter: rnewman, Unassigned)

References

Details

(Keywords: meta)

33: 494,460 bytes of PNGs. 37: 1,367,991 bytes of PNGs. Some of this will go away when we remove the old tablet resources, but I'm very interested to know where the remaining ~1MB came from!
Looks like we ship lots of Google Play Services assets… including stuff like G+ signin and "+1" buttons, the Play Services icon, and a bunch of actionbar icons. This can definitely be improved. Wes, Chris, do you have any thoughts here?
Blocks: fatfennec
Flags: needinfo?(wjohnston)
Flags: needinfo?(chriskitching)
tracking-fennec: --- → ?
I saw this when I glanced around: http://tools.android.com/tech-docs/new-build-system/resource-shrinking Note, that requires that we proguard play services (or any libs we want to remove resources from), but I think we do that now anyway.
Flags: needinfo?(wjohnston)
Adding another dependency that should help.
Depends on: 1115004
tracking-fennec: ? → ---
Depends on: 1115000
Summary: Significant resource space regression between 33 and 37 → [meta] Significant resource space regression between 33 and 37
tracking-fennec: --- → ?
Flags: needinfo?(chriskitching)
(In reply to Richard Newman [:rnewman] from comment #0) > Some of this will go away when we remove the old tablet resources, but I'm > very interested to know where the remaining ~1MB came from! I just landed part 1 of bug 1106935 which removes the duplicated old tablet resources - however, it's only 4 or so pngs. Feel free to update the recorded size (e.g. comment 0). We did add some non-duplicated resources with new tablet (notably with xxhdpi sizing - old tablet did not use xxhdpi), which will have contributed to the increase.
This is a meta bug. If you want tracking nominate the individual bugs that are concrete improvements.
tracking-fennec: ? → -
Keywords: meta
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.