Closed Bug 402407 Opened 13 years ago Closed 13 years ago

PNGs can be re-compressed further.

Categories

(Calendar :: General, enhancement)

enhancement
Not set
normal

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: gkw, Assigned: gkw)

Details

(Whiteboard: [gdata-0.4])

Attachments

(2 files, 3 obsolete files)

Had some time (again) to scrutinize the calendar folder of the 0.7 release source to re-compress the PNGs. Used PNGGauntlet with brute-force option.

-----------
Total sizes
-----------
Before: 768,574 bytes
After: 724,350 bytes

Decreased by 44,224 bytes, >5%. Not *that* significant, but worth considering.
Flags: wanted-calendar0.8?
all 3 attachments come with intact original folder hierarchy.
It would be good to have this in 0.8
Flags: wanted-calendar0.8? → wanted-calendar0.8+
Gary, are the files still up to date? If so, please request ui-review from christian so we can check in.
Assignee: nobody → nth10sd
Attachment #287290 - Attachment is obsolete: true
Status: NEW → ASSIGNED
Attachment #297533 - Flags: ui-review?(christian.jansen)
I'm quite sure that all hierarchy has been retained.
Attachment #287291 - Attachment is obsolete: true
Attachment #287293 - Attachment is obsolete: true
Attachment #297535 - Flags: ui-review?(christian.jansen)
For attachments in comment#6 and comment#7 , I had updated to latest cvs as of 16 Jan 2008, then re-compressed PNGs using PNGGauntlet with brute-force option.

Total size of all PNGs in calendar folder:
Before: 159 files - 677,033 bytes
After: 159 files - 609,730 bytes

67,303 bytes saved, that's >65kB (~10%) decrease.
the above-mentioned attachments were from a trunk cvs checkout though. i'll do up more attachments for the mozilla1.8 branch.
My bad, the files for mozilla1.8 branch would be similar to the ones above.
Comment on attachment 297535 [details]
7z file containing png files in the other folders

r=christian
Attachment #297535 - Flags: ui-review?(christian.jansen) → ui-review+
Attachment #297533 - Flags: ui-review?(christian.jansen) → ui-review+
Comment on attachment 297533 [details]
7z file containing png files in base/

r=christian
Should this be checked in with -kb flag?
Keywords: checkin-needed
(In reply to comment #13)
Yes.
checked in pngs on trunk and MOZILLA_1_8_BRANCH
=>FIXED
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Keywords: checkin-needed
Target Milestone: --- → 0.8
Status: RESOLVED → VERIFIED
Whiteboard: [gdata-cvs]
Some stats on the effect of this checkin:

20080210 Sb Win .zip - 7,136,031 bytes
20080211 Sb Win .zip - 7,128,823 bytes
20080210 Sb Linux - 8,721,341 bytes
20080211 Sb Linux - 8,711,330 bytes
20080210 Sb Mac - 14,595,289 bytes
20080211 Sb Mac - 14,588,935 bytes
(~0.1% decrease for Win & Linux, ~0.05% decrease for Mac)

20080210 Lightning Win - 1,051,403 bytes
20080211 Lightning Win - 1,046,184 bytes
20080210 Lightning Linux - 1,089,075 bytes
20080211 Lightning Linux - 1,083,848 bytes
20080210 Lightning Mac - 1,222,289 bytes
20080211 Lightning Mac - 1,216,356 bytes
(~0.5% decrease for all platforms)

http://bonsai.mozilla.org/cvsquery.cgi?treeid=default&module=CalendarClient&branch=MOZILLA_1_8_BRANCH&branchtype=match&dir=&file=&filetype=match&who=&whotype=match&sortby=Date&hours=2&date=explicit&mindate=2008-02-11+00%3A00%3A00&maxdate=2008-02-11+23%3A59%3A59&cvsroot=%2Fcvsroot
Whiteboard: [gdata-cvs] → [gdata-0.4]
You need to log in before you can comment on or make changes to this bug.