Closed Bug 1193580 Opened 8 years ago Closed 8 years ago

Optimize icons

Categories

(Firefox for Android Graveyard :: General, defect)

Product:
Firefox for Android Graveyard
Component:
General
Platform:
ARM
Android
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(firefox43 fixed)

Status:
RESOLVED FIXED
Milestone:
Firefox 43
Tracking Flags:
Tracking Status
firefox43 --- fixed

People

(Reporter: liuche, Assigned: mcomella)

References

Details

Attachments

(1 file)

MozReview Request: Bug 1193580 - Losslessly compress png assets. r=liuche
40 bytes, text/x-review-board-request
liuche
: review+
Details 
I didn't realize I should optimize the new doorhanger/urlbar images.
Just talked to mcomella and apparently he's doing this for all the icons, so renaming this bug.
Assignee: nobody → michael.l.comella
Summary: Optimize tracking protection and mixed content icons → Optimize icons
No longer blocks: tp-v1
Like bug 1150974 before it. Notably I used trimage before, rather than ImageOptim, so we may get some wins from using additional algos.

ImageOptim claims ~87KB improvement but my filesytem sees more on the resultant APK (42897147k - 42895727k = 1420k)
Depends on: 1150974
Comment on attachment 8646666 [details]
MozReview Request: Bug 1193580 - Losslessly compress png assets. r=liuche

https://reviewboard.mozilla.org/r/15825/#review14109

r+, if you also compress the images in mobile/android/themes/ and mobile/android/branding
Attachment #8646666 - Flags: review?(liuche) → review+
url:        https://hg.mozilla.org/integration/fx-team/rev/1c1d97b563f410d571e4925dab3f53fcae6c5272
changeset:  1c1d97b563f410d571e4925dab3f53fcae6c5272
user:       Michael Comella <michael.l.comella@gmail.com>
date:       Tue Aug 11 16:52:38 2015 -0700
description:
Bug 1193580 - Losslessly compress png assets. r=liuche
10kb saved :)
https://hg.mozilla.org/mozilla-central/rev/1c1d97b563f4
Status: NEW → RESOLVED
Closed: 8 years ago
status-firefox43: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 43
This will make be redo (via rebase) pretty much every file in bug 1150742, which will take hours so I'm going to back this out, rerun the algorithm, and reland it after I land bug 1150742.
url:        https://hg.mozilla.org/integration/fx-team/rev/a9edaaee5c81016239e9383c86568f1666d860a4
changeset:  a9edaaee5c81016239e9383c86568f1666d860a4
user:       Michael Comella <michael.l.comella@gmail.com>
date:       Thu Aug 13 10:27:24 2015 -0700
description:
Bug 1193580 - Backout changeset 1c1d97b563f410d571e4925dab3f53fcae6c5272.

This is temporary and only to easy the landing of bug 1150742.
(Added as a dep because it could make future backouts/changes messy)
Depends on: 1150742
url:        https://hg.mozilla.org/integration/fx-team/rev/bdb441abebf03c379b538e388f16f722e7fefa96
changeset:  bdb441abebf03c379b538e388f16f722e7fefa96
user:       Michael Comella <michael.l.comella@gmail.com>
date:       Thu Aug 13 10:43:51 2015 -0700
description:
Bug 1193580 - Losslessy compress png assets. r=liuche

Note that this not the originally reviewed patch in the bug, but a re-run of
the algorithm that produced that patch.
You mention that you ran ImageOptim here, rather than trimage...care to share the command you ran this time?
(In reply to Nathan Toone from comment #13)
> You mention that you ran ImageOptim here, rather than trimage...care to
> share the command you ran this time?

I used the GUI as the ImageOptim CL is verbose.
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.