X11: Use better visuals for OpenGL

ui/gl/gl_context_glx.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "ui/gl/gl_context_glx.h"
 
 extern "C" {
 #include <X11/Xlib.h>
 }
 #include <memory>
@@ skipping 148 matching lines @@
   return display_;
 }
 
 bool GLContextGLX::Initialize(
     GLSurface* compatible_surface, GpuPreference gpu_preference) {
   display_ = static_cast<XDisplay*>(compatible_surface->GetDisplay());
 
   GLXContext share_handle = static_cast<GLXContext>(
       share_group() ? share_group()->GetHandle() : nullptr);
 
+  int fbconfigid = 0;
+  glXGetFBConfigAttrib(
+      display_, static_cast<GLXFBConfig>(compatible_surface->GetConfig()),
+      GLX_FBCONFIG_ID, &fbconfigid);

[piman, 2016/09/22 21:24:55]

Is this used anywhere?

[Tom (Use chromium acct), 2016/09/23 20:00:37]

Removed.  More leftover debugging..

+
   if (GLSurfaceGLX::IsCreateContextSupported()) {
     DVLOG(1) << "GLX_ARB_create_context supported.";
     context_ = CreateHighestVersionContext(
         display_, static_cast<GLXFBConfig>(compatible_surface->GetConfig()),
         share_handle);
     if (!context_) {
       LOG(ERROR) << "Failed to create GL context with "
                  << "glXCreateContextAttribsARB.";
       return false;
     }
@@ skipping 133 matching lines @@
 
 bool GLContextGLX::WasAllocatedUsingRobustnessExtension() {
   return GLSurfaceGLX::IsCreateContextRobustnessSupported();
 }
 
 GLContextGLX::~GLContextGLX() {
   Destroy();
 }
 
 }  // namespace gl