<chromium> Only enable robustness if the shared context is using it

Chromium currently assumes that all GLContexts are checking the availability
of robustness on the system through the same code path. Since we're passing
it a context created by Qt, and that Qt never uses
EGL_EXT_create_context_robustness, the context creation fails when trying to
share a non-robustness-enabled context with one that does.

Fix the issue by asking the shared GLContext if it was allocated using the
extension, Chromium's contexts currently always return true and the Qt context
will return false. This relies on the fact that the Qt context is always the
first one added to the ShareGroup and that it is going to be the one queried,
but this should be reasonably reliable.

Change-Id: I6456429db678f077cd8a72032711361da01e72ea
Reviewed-by: Andras Becsi <[email protected]>

Author: jturcotte

chromium/ui/gl/gl_context_egl.cc

@@ -52,7 +52,8 @@ bool GLContextEGL::Initialize(
   config_ = compatible_surface->GetConfig();
 
   const EGLint* context_attributes = NULL;
-  if (GLSurfaceEGL::IsCreateContextRobustnessSupported()) {
+  if (GLSurfaceEGL::IsCreateContextRobustnessSupported()
+      && (!share_group()->GetContext() || share_group()->GetContext()->WasAllocatedUsingRobustnessExtension())) {
     DVLOG(1) << "EGL_EXT_create_context_robustness supported.";
     context_attributes = kContextRobustnessAttributes;
   } else {

chromium/ui/gl/gl_context_glx.cc

@@ -53,7 +53,8 @@ bool GLContextGLX::Initialize(
   if (GLSurfaceGLX::IsCreateContextSupported()) {
     DVLOG(1) << "GLX_ARB_create_context supported.";
     std::vector<int> attribs;
-    if (GLSurfaceGLX::IsCreateContextRobustnessSupported()) {
+    if (GLSurfaceGLX::IsCreateContextRobustnessSupported()
+        && (!share_group()->GetContext() || share_group()->GetContext()->WasAllocatedUsingRobustnessExtension())) {
       DVLOG(1) << "GLX_ARB_create_context_robustness supported.";
       attribs.push_back(GLX_CONTEXT_RESET_NOTIFICATION_STRATEGY_ARB);
       attribs.push_back(GLX_LOSE_CONTEXT_ON_RESET_ARB);