Bug 55878 – [Gtk] toDataURL uses incorrect quality value when saving GdkPixbuf to buffer

• Status: RESOLVED
• Resolution: FIXED
• Component: Canvas

[Zan Dobersek]

Gtk uses ImageBuffer::toDataURL in ImageBufferGtk.cpp to get data URL for a buffer. If data URL should be of mimetype image/jpeg, current code uses quality parameter if passed and utilizes it when saving a GdkPixbuf (which we get from the cairo surface) to a buffer.

The problem is that quality passed to the gdk_pixbuf_save_to_buffer function should be in range of [0, 100] while we currently keep it in range [0.0, 1.0]. This results in canvas/philip/tests/toDataURL.jpeg.quality.basic.html failing.

[Zan Dobersek]

Created attachment 84944 [details]
Put quality into an appropriate range.

This patch multiplies quality variable, which is originally in range [0.0, 1.0] by 100.0 to ensure it is in the correct range of [0, 100], as seen in gdk_pixbuf_save function documentation[1].

[1] http://library.gnome.org/devel/gdk-pixbuf/2.22/gdk-pixbuf-file-saving.html#gdk-pixbuf-save

[Martin Robinson]

Comment on attachment 84944 [details]
Put quality into an appropriate range.

Nice catch.

[WebKit Commit Bot]

Comment on attachment 84944 [details]
Put quality into an appropriate range.

Clearing flags on attachment: 84944

Committed r80546: <http://trac.webkit.org/changeset/80546>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.