[webkit-reviews] review denied: [Bug 16378] Implement Icon for Gtk : [Attachment 17818] Implement Icon for Gtk

[bugzilla-daemon at webkit.org]

Alp Toker <alp at atoker.com> has denied Christian Dywan
<christian at twotoasts.de>'s request for review:
Bug 16378: Implement Icon for Gtk
http://bugs.webkit.org/show_bug.cgi?id=16378

Attachment 17818: Implement Icon for Gtk
http://bugs.webkit.org/attachment.cgi?id=17818&action=edit

------- Additional Comments from Alp Toker <alp at atoker.com>
I didn't check the icon name code yet, but the drawing should almost certainly
be done with cairo and gdk_cairo_set_source_pixbuf() rather than
gdk_draw_pixbuf() so that the drawing operation will integrate with the rest of
the graphics backend.

A ChangeLog, particularly to explain lookupIconName() (which was non-obvious to
me) would be helpful.

Thanks!