[webkit-reviews] review requested: [Bug 32763] [Chromium] Eliminate redundant data copy in GraphicsContext3D readback : [Attachment 45219] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Fri Dec 18 19:09:30 PST 2009


Kenneth Russell <kbr at google.com> has asked  for review:
Bug 32763: [Chromium] Eliminate redundant data copy in GraphicsContext3D
readback
https://bugs.webkit.org/show_bug.cgi?id=32763

Attachment 45219: Patch
https://bugs.webkit.org/attachment.cgi?id=45219&action=review

------- Additional Comments from Kenneth Russell <kbr at google.com>
Eliminated useless creation of a CGBitmapContext and from there a CGImage. Now
create CGImage directly from data. Also changed readPixels parameters and
CGImage alpha info to try to avoid byte swapping, and disabled interpolation
when drawing the GL content. Some test cases run twice as fast with these
changes, though more work is needed to achieve desired performance.

No test case; performance optimization only. Ran WebGL demos from Khronos site
to verify changes.


More information about the webkit-reviews mailing list