RESOLVED FIXED 92301
[BlackBerry] readPixels on FBOs are limited to canvas size on SGX platforms
https://bugs.webkit.org/show_bug.cgi?id=92301
Summary [BlackBerry] readPixels on FBOs are limited to canvas size on SGX platforms
Joshua Netterfield
Reported 2012-07-25 14:25:38 PDT
We use a workaround for readPixels because of some problems the SGX driver has with reading pixels. This workaround is currently limited to the size of the canvas.
Attachments
Patch (2.49 KB, patch)
2012-07-25 16:42 PDT, Joshua Netterfield
no flags
Patch (2.87 KB, patch)
2012-07-26 06:45 PDT, Joshua Netterfield
no flags
Joshua Netterfield
Comment 1 2012-07-25 16:42:26 PDT
Rob Buis
Comment 2 2012-07-25 19:32:49 PDT
Comment on attachment 154479 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=154479&action=review > Source/WebCore/ChangeLog:8 > + This fixes an existing test (uninitialized-test), no new tests. Better state which one. > Source/WebCore/platform/graphics/opengl/GraphicsContext3DOpenGLES.cpp:120 > + readPixels(x, y, width, height, GL_RGBA, GL_UNSIGNED_BYTE, pixels); This seems not BB specific? Is it safe to do?
Joshua Netterfield
Comment 3 2012-07-26 06:43:46 PDT
Right. Forgot about the anti-aliasing stuff on other platforms. I wonder if it's intentionally not in readPixelsAndConvertToBGRAIfNecessary...
Joshua Netterfield
Comment 4 2012-07-26 06:45:25 PDT
Rob Buis
Comment 5 2012-07-26 07:04:57 PDT
Comment on attachment 154642 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=154642&action=review Better :) > Source/WebCore/platform/graphics/opengl/GraphicsContext3DOpenGLES.cpp:127 > + ::glReadPixels(x, y, width, height, GL_RGBA, GL_UNSIGNED_BYTE, pixels); You do the same thing twice, could add a helper method for that, but as we discussed can be done later.
WebKit Review Bot
Comment 6 2012-07-26 09:34:51 PDT
Comment on attachment 154642 [details] Patch Clearing flags on attachment: 154642 Committed r123759: <http://trac.webkit.org/changeset/123759>
WebKit Review Bot
Comment 7 2012-07-26 09:34:55 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.