I notice that many images in the modern theme (didn't check classic) are transparent, and need not be. Transparency imposes significant runtime cost, especially for images that must be tiled (at least on Mac). Some examples: mozilla/themes/modern/navigator/icons/btn1.gif mozilla/themes/modern/global/toolbar/tb-mid.gif (gets tiled!) also mozilla/themes/modern/global/preview.gif We should probably make a pass through all skin images and ensure that transparency is off when possible.
triage approving performance win
i looked at those images in photoshop and they are not transparent
The GIFs have transparency.
Note: I used the GIF Analyzer, from <ftp://ftp.totallyhip.com/products/GIFUtilityPack/GIFUtilityPack10FAT.bin> to look at the image. It shows that the 'btn1.gif' is transparent, with transparency index 235. So even if PhotoShop shows no transparency, however you export to .gif is adding transparency, needlessly.
Created attachment 70167 [details] turning transparency off for those 3 images going to go through other images now
be careful - some images need to be transparent as far as I know.
Created attachment 70197 [details] classic win preview.gif doesn't look like anything else in modern or classic has transparency if it doesn't need it
sr=hewitt on all the images
r=andreww for images listed in this bug.
Verified on all platforms (commercial build: 2002-02-25-08-trunk)