Closed
Bug 1193580
Opened 8 years ago
Closed 8 years ago
Optimize icons
Categories
(Firefox for Android Graveyard :: General, defect)
Tracking
(firefox43 fixed)
RESOLVED
FIXED
Firefox 43
Tracking | Status | |
---|---|---|
firefox43 | --- | fixed |
People
(Reporter: liuche, Assigned: mcomella)
References
Details
Attachments
(1 file)
I didn't realize I should optimize the new doorhanger/urlbar images.
Reporter | ||
Comment 1•8 years ago
|
||
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
Assignee | ||
Comment 2•8 years ago
|
||
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
Assignee | ||
Comment 3•8 years ago
|
||
Bug 1193580 - Losslessly compress png assets. r=liuche
Attachment #8646666 -
Flags: review?(liuche)
Reporter | ||
Comment 4•8 years ago
|
||
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+
Assignee | ||
Comment 5•8 years ago
|
||
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
Reporter | ||
Comment 6•8 years ago
|
||
10kb saved :)
Comment 7•8 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/1c1d97b563f4
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 43
Assignee | ||
Comment 8•8 years ago
|
||
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.
Assignee | ||
Comment 9•8 years ago
|
||
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.
Assignee | ||
Comment 10•8 years ago
|
||
(Added as a dep because it could make future backouts/changes messy)
Depends on: 1150742
Assignee | ||
Comment 11•8 years ago
|
||
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.
https://hg.mozilla.org/mozilla-central/rev/a9edaaee5c81 https://hg.mozilla.org/mozilla-central/rev/bdb441abebf0
Comment 13•8 years ago
|
||
You mention that you ran ImageOptim here, rather than trimage...care to share the command you ran this time?
Assignee | ||
Comment 14•8 years ago
|
||
(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.
Updated•3 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•