Closed Bug 1193580 Opened 5 years ago Closed 5 years ago
40 bytes, text/x-review-board-request
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
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 <email@example.com> date: Tue Aug 11 16:52:38 2015 -0700 description: Bug 1193580 - Losslessly compress png assets. r=liuche
10kb saved :)
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 <firstname.lastname@example.org> 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 <email@example.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.
You need to log in before you can comment on or make changes to this bug.