Bug 43754 – Use Chromium OpenGL bindings rather than GLEW in WebGraphicsContextDefaultImpl

• Component: WebGL
• Status: RESOLVED FIXED

[Kenneth Russell]

In the Chromium WebGL port, the "default" implementation of WebGraphicsContext3D calls OpenGL directly, currently via GLEW. We need to switch this to use Chromium's OpenGL bindings rather than GLEW so that Chromium's test shell can use the OSMesa renderer. This will allow WebGL tests to be run on the build bots all of the time.

[Kenneth Russell]

Created attachment 64032 [details]
Patch

From the ChangeLog:

Deleted per-platform OpenGL context management code, now abstracted via GLContext. Built and tested in Chromium with --in-process-webgl flag.

Thanks to Al Patrick for the initial patch.

[Kenneth Russell]

Created attachment 64033


From the ChangeLog:

Deleted per-platform OpenGL context management code, now abstracted via GLContext. Built and tested in Chromium with --in-process-webgl flag.

Thanks to Al Patrick for the initial patch.

[WebKit Review Bot]

Attachment 64032 [details] did not pass style-queue:

Failed to run "['WebKitTools/Scripts/check-webkit-style']" exit_code: 1
WebKit/chromium/src/WebGraphicsContext3DDefaultImpl.cpp:35:  Found other header before a header this file implements. Should be: config.h, primary header, blank line, and then alphabetically sorted.  [build/include_order] [4]
WebKit/chromium/src/WebGraphicsContext3DDefaultImpl.cpp:72:  Use 0 instead of NULL.  [readability/null] [5]
WebKit/chromium/src/WebGraphicsContext3DDefaultImpl.cpp:107:  Use 0 instead of NULL.  [readability/null] [5]
Total errors found: 3 in 4 files


If any of these errors are false positives, please file a bug against check-webkit-style.

[Dimitri Glazkov (Google)]

Comment on attachment 64032 [details]
Patch

r=me, after style fixes:

WebKit/chromium/src/WebGraphicsContext3DDefaultImpl.cpp:72
 +      , m_glContext(NULL)
(0)

WebKit/chromium/src/WebGraphicsContext3DDefaultImpl.cpp:98
 +          delete m_glContext;
Maybe make OwnPtr?

WebKit/chromium/src/WebGraphicsContext3DDefaultImpl.cpp:107
 +      m_glContext = gfx::GLContext::CreateOffscreenGLContext(NULL);
(0)

WebKit/chromium/src/WebGraphicsContext3DDefaultImpl.cpp:427
 +      // TODO(apatrick): OpenGL ES 2 does not support GL_BGRA so this fails when
FIXME:

[Kenneth Russell]

(In reply to comment #4)
> (From update of attachment 64032 [details])
> r=me, after style fixes:
> 
> WebKit/chromium/src/WebGraphicsContext3DDefaultImpl.cpp:72
>  +      , m_glContext(NULL)
> (0)
> 
> WebKit/chromium/src/WebGraphicsContext3DDefaultImpl.cpp:98
>  +          delete m_glContext;
> Maybe make OwnPtr?
> 
> WebKit/chromium/src/WebGraphicsContext3DDefaultImpl.cpp:107
>  +      m_glContext = gfx::GLContext::CreateOffscreenGLContext(NULL);
> (0)
> 
> WebKit/chromium/src/WebGraphicsContext3DDefaultImpl.cpp:427
>  +      // TODO(apatrick): OpenGL ES 2 does not support GL_BGRA so this fails when
> FIXME:

Will fix these issues before commit.

[Kenneth Russell]

Committed r65093: <http://trac.webkit.org/changeset/65093>

[WebKit Review Bot]

http://trac.webkit.org/changeset/65093 might have broken Chromium Linux Release