[webkit-reviews] review denied: [Bug 117868] Texture unit code refactoring : [Attachment 205172] Texture unit code refactoring
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Wed Jun 26 06:35:50 PDT 2013
Noam Rosenthal <noam at webkit.org> has denied Przemyslaw Szymanski
<p.szymanski3 at samsung.com>'s request for review:
Bug 117868: Texture unit code refactoring
https://bugs.webkit.org/show_bug.cgi?id=117868
Attachment 205172: Texture unit code refactoring
https://bugs.webkit.org/attachment.cgi?id=205172&action=review
------- Additional Comments from Noam Rosenthal <noam at webkit.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=205172&action=review
R- due to missing changelog.
> Source/WebCore/html/canvas/WebGLRenderingContext.cpp:-612
> - m_textureUnits[i].m_texture2DBinding = 0;
4 spaces for indent please
> Source/WebCore/html/canvas/WebGLRenderingContext.cpp:4811
> + const GC3Denum target =
m_textureUnits[ii].m_textureBinding->getTarget();
We don't use local const in webkit.
More information about the webkit-reviews
mailing list