Compiling GraphicsContext3DOpenGLCommon.cpp on WinCairo fails, because GL_HALF_FLOAT_ARB is undefined. It's defined in <GL/glext.h>, but when I include that, I get other compile errors.
Created attachment 218655 [details] Patch
Comment on attachment 218655 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=218655&action=review > Source/WebCore/platform/graphics/opengl/GraphicsContext3DOpenGLCommon.cpp:1368 > +#if !PLATFORM(WIN) || defined(GL_HALF_FLOAT_ARB) Can you file a bug for the break in a Windows with GLext.h included, and add a FIXME here? Then I'll r+ this as a build fix.
Created attachment 218782 [details] Patch
On second thought, I think it's better to define GL_HALF_FLOAT_ARB when OPENGL_ES_2 is used. It doesn't seem we're meant to include <GL/glext.h> when OPENGL_ES_2 is used, rather <GLES2/gl2ext.h>.
Comment on attachment 218782 [details] Patch Clearing flags on attachment: 218782 Committed r160324: <http://trac.webkit.org/changeset/160324>
All reviewed patches have been landed. Closing bug.
*** Bug 125464 has been marked as a duplicate of this bug. ***
Oops, I filed this patch for the same issue. I think it affects all GL ES 2.0 targets, not just Windows. https://bugs.webkit.org/show_bug.cgi?id=125541 I'll update my patch to weaken the #if guard to just #if USE(OPENGL_ES_2).
(In reply to comment #8) > Oops, I filed this patch for the same issue. I think it affects all GL ES 2.0 targets, not just Windows. > > https://bugs.webkit.org/show_bug.cgi?id=125541 > > I'll update my patch to weaken the #if guard to just #if USE(OPENGL_ES_2). Oops, actually that would be wrong; I didn't realize that HALF_FLOAT_OES and HALF_FLOAT_ARB are different values -- an actual GLES 2 implementation wants HALF_FLOAT_OES (0x8D61). I think my other patch is still OK, though it's weird to have guards around the same thing in two places. Does the Windows GL ES 2 implementation really want HALF_FLOAT_ARB?
If you're using the ANGLE in the WebKit tree then it will bail in glTexImage2D if you use the GL_HALF_FLOAT_ARB constant (look in src/libGLESv2/libGLESv2.cpp for HALF_FLOAT_OES). It doesn't seem to support half float vertices at all. Maybe you're using some vendor OpenGL ICD, in which case it'd be fine. I'm not sure how the Windows build is set up. Sorry for so many comments.