[Webkit-unassigned] [Bug 70293] [TexMap][QT] TexMapGL renders a strange one-pixel border of BitmapTexture.

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Mon Oct 17 18:34:38 PDT 2011


https://bugs.webkit.org/show_bug.cgi?id=70293


Noam Rosenthal <noam.rosenthal at nokia.com> changed:

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




--- Comment #4 from Noam Rosenthal <noam.rosenthal at nokia.com>  2011-10-17 18:34:38 PST ---
(From update of attachment 111363)
View in context: https://bugs.webkit.org/attachment.cgi?id=111363&action=review

Good catch!
Some nitpicks and we're done.

> Source/WebCore/ChangeLog:9
> +        It is because BitmapTexture is larger than original bitmap due to POT and the
> +        rest space of BitmapTexture would be filled undefined values.

Grammar:
Bug occurs because BitmapTexture is larger than the content due to rounding to NPOT, and its pixel data is never initialized.

> Source/WebCore/platform/graphics/opengl/TextureMapperGL.cpp:507
> +static void texImage2DResourceSafe(size_t width, size_t height)

Don't like the name... how about resetTexturePixelData?

> Source/WebCore/platform/graphics/opengl/TextureMapperGL.cpp:511
> +    if (width > 0 && height > 0) {

those are unsigned, so 
if (width && height)

-- 
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