Bug 91600

Summary:

[Qt] Modify the using of the QImage::Format enum to the appropriate functions from NativeImageQt

Product:

WebKit

Reporter:

Zoltan Horvath <zoltan>

Component:

WebCore Misc.

Assignee:

Zoltan Horvath <zoltan>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

cmarcelo, menard, noam, webkit.review.bot

Priority:

Version:

528+ (Nightly build)

Hardware:

Unspecified

OS:

All

Attachments:

Description	Flags
Patch	none

Zoltan Horvath

Reported 2012-07-18 01:08:11 PDT

Use NativeImageQt::defaultFormatForAlphaEnabledImages() and NativeImageQt::defaultFormatForOpaqueImages() instead of the direct imagetypes at the appropriate places.

Attachments
Patch (6.81 KB, patch) 2012-07-18 01:21 PDT, Zoltan Horvath	no flags	Details Formatted Diff Diff
View All Add attachment proposed patch, testcase, etc.

Zoltan Horvath

Comment 1 2012-07-18 01:21:31 PDT

Created attachment 152960 [details] Patch

Andreas Kling

Comment 2 2012-07-18 05:45:06 PDT

Comment on attachment 152960 [details] Patch LGTM.

WebKit Review Bot

Comment 3 2012-07-18 05:50:51 PDT

Comment on attachment 152960 [details] Patch Clearing flags on attachment: 152960 Committed r122953: <http://trac.webkit.org/changeset/122953>

WebKit Review Bot

Comment 4 2012-07-18 05:50:57 PDT

All reviewed patches have been landed. Closing bug.

Note You need to log in before you can comment on or make changes to this bug.