Bug 113784 – [Qt] Fail gracefully if an OpenGL context could not be created.

• Component: New Bugs
• Status: RESOLVED FIXED

[Zeno Albisser]

[Qt] Fail gracefully if an OpenGL context could not be created.

[Zeno Albisser]

Created attachment 196143 [details]
Patch

[Jocelyn Turcotte]

Comment on attachment 196143 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=196143&action=review

> Source/WebCore/platform/graphics/qt/GraphicsContext3DQt.cpp:373
> -    return context->m_private ? context.release() : 0;
> +    return (context->m_private && context->platformGraphicsContext3D()) ? context.release() : 0;

To avoid spreading the failure logic I think it would be cleaner to add a check in GraphicsContext3D::GraphicsContext3D and set m_private = nullptr if (!m_private->m_platformContext).

Something like changing "if (!m_private->m_surface) {" to "if (!m_private->m_surface || !m_private->m_platformContext) {" at line 398.

[Zeno Albisser]

Created attachment 196205 [details]
Patch

[Zeno Albisser]

(In reply to comment #2)
> To avoid spreading the failure logic I think it would be cleaner to add a check in GraphicsContext3D::GraphicsContext3D and set m_private = nullptr if (!m_private->m_platformContext).

A very good point! Thx a lot! :)

[Zeno Albisser]

Comment on attachment 196205 [details]
Patch

Clearing flags on attachment: 196205

Committed r147553: <http://trac.webkit.org/changeset/147553>

[Zeno Albisser]

All reviewed patches have been landed.  Closing bug.