Justin Dolske <firstname.lastname@example.org> has asked Robert O'Callahan (:roc)
<email@example.com> for superreview:
Bug 410215: GTK's .xpm decoding is weird, stop using XPM as the default window
Attachment 295160: Patch v.1 for review
------- Additional Comments from Justin Dolske <firstname.lastname@example.org>
This changes the icon setting code to prefer a PNG, and if no PNG is found it
falls back to looking for an XPM. This allow us to start switching apps over to
PNG without forcing everyone to change at once.
I went ahead and rolled in support for additional icon sizes, which would be
needed by bug 404402. I only added the sizes recommended by:
There's overhead for each size queried -- looks like we stat() in each
extension's directory, each time a window is opened. So, we probably don't want
this list to include every conceivable icon size.
SetDefaultIcon() seemed to be needlessly duplicating code, so I just have it
use SetIcon() now too.
Tested on Solaris, confirmed the XPM fallback works and that when multiple
icons are count GTK seems to prefer the right size.