Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

optimize png images using zopflipng #7478

Merged
merged 1 commit into from
Jul 10, 2016
Merged

optimize png images using zopflipng #7478

merged 1 commit into from
Jul 10, 2016

Conversation

PeterDaveHello
Copy link
Contributor

@PeterDaveHello PeterDaveHello commented Jul 8, 2016

Use Google's zopflipng to re-compress png images losslessly as below:

 extensions/b2g/images/div_line_left.png           | Bin 174 -> 169 bytes
 extensions/b2g/images/[email protected]      | Bin 196 -> 185 bytes
 extensions/b2g/images/[email protected]        | Bin 319 -> 295 bytes
 extensions/b2g/images/div_line_right.png          | Bin 173 -> 166 bytes
 extensions/b2g/images/[email protected]     | Bin 193 -> 184 bytes
 extensions/b2g/images/[email protected]       | Bin 319 -> 295 bytes
 extensions/b2g/images/document_bg.png             | Bin 609 -> 560 bytes
 extensions/b2g/images/icon_next_page.png          | Bin 325 -> 310 bytes
 extensions/b2g/images/[email protected]     | Bin 343 -> 338 bytes
 extensions/b2g/images/icon_previous_page.png      | Bin 384 -> 372 bytes
 extensions/b2g/images/[email protected] | Bin 396 -> 395 bytes
 extensions/b2g/images/[email protected]       | Bin 1229 -> 1170 bytes
 extensions/b2g/images/[email protected]      | Bin 1090 -> 1051 bytes
 extensions/b2g/images/spinner.png                 | Bin 3225 -> 2759 bytes
 extensions/b2g/images/toolbar_background.png      | Bin 4695 -> 4657 bytes
 extensions/chromium/icon128.png                   | Bin 3463 -> 2989 bytes
 extensions/chromium/icon16.png                    | Bin 622 -> 594 bytes
 extensions/chromium/icon19.png                    | Bin 712 -> 679 bytes
 extensions/chromium/icon38.png                    | Bin 1330 -> 1300 bytes
 extensions/chromium/icon48.png                    | Bin 1691 -> 1671 bytes
 extensions/firefox/icon.png                       | Bin 1612 -> 1597 bytes
 extensions/firefox/icon64.png                     | Bin 2089 -> 2003 bytes
 web/images/texture.png                            | Bin 2418 -> 2417 bytes
 web/images/toolbarButton-menuArrows.png           | Bin 108 -> 107 bytes
 web/images/[email protected]   | Bin 871 -> 859 bytes
 web/images/[email protected]     | Bin 220 -> 219 bytes
 web/images/treeitem-collapsed-rtl.png             | Bin 183 -> 143 bytes
 web/images/[email protected]          | Bin 205 -> 167 bytes

@timvandermeij
Copy link
Contributor

There is a problem with web/images/loading-small.png and web/images/[email protected]. Those are animated PNGs, but this patch removed the animation. We might need to exclude these two or somehow let zopflipng know that it is an animated PNG. Could you look into that?

@PeterDaveHello
Copy link
Contributor Author

Okay, I'll take a look at this problem.

@PeterDaveHello
Copy link
Contributor Author

Problem confirmed, reported to zopflipng upstream here: google/zopfli#118, rollback those two images to origin. Thanks @timvandermeij.

@timvandermeij
Copy link
Contributor

/botio-linux preview

@pdfjsbot
Copy link

From: Bot.io (Linux)


Received

Command cmd_preview from @timvandermeij received. Current queue size: 0

Live output at: http://107.21.233.14:8877/8c91f1639cd07dd/output.txt

@timvandermeij timvandermeij merged commit 1695893 into mozilla:master Jul 10, 2016
@timvandermeij
Copy link
Contributor

Thank you!

@PeterDaveHello PeterDaveHello deleted the image-optimize branch July 10, 2016 12:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants