[Webkit-unassigned] [Bug 40884] [Qt] When we render WebGL offscreen, color conversion cost a lot of CPU cycles

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Jul 1 16:20:53 PDT 2010


Andreas Kling <andreas.kling at nokia.com> changed:

           What    |Removed                     |Added
  Attachment #59197|review?, commit-queue?      |review-, commit-queue-
               Flag|                            |

--- Comment #2 from Andreas Kling <andreas.kling at nokia.com>  2010-07-01 16:20:53 PST ---
(From update of attachment 59197)
Unnecessary empty line.

 +      // GL give us ABGR on 32 bits, and with the origin at the bottom left

 +      free(m_pixels);
WebKit normally uses new/delete, is there a specific reason for malloc/free here?

 +  static inline quint32 swapPixelsRGB32(quint32 pixel)
I don't like this function name - it's swapping color values, not pixels.

 +          format = QImage::Format_ARGB32_Premultiplied;
 +          format = QImage::Format_RGB32;
You're setting format twice here. It ends up being RGB32 which is incorrect for this code path.

Looks good otherwise, r- for the wrong format in the alpha channel case.

Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

More information about the webkit-unassigned mailing list