fix for GTK assertion from gtk_widget_set_colormap, bug 1882040

git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@51481 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775

diff --git a/src/gtk/glcanvas.cpp b/src/gtk/glcanvas.cpp
index 909c5c02e20098aec37d4f149a00b2663ef1c07e..cb261d2702986ec01e4b96856b5f14f8a0bdcf22 100644 (file)
--- a/src/gtk/glcanvas.cpp
+++ b/src/gtk/glcanvas.cpp
@@ -100,6 +100,34 @@ gtk_glcanvas_size_callback(GtkWidget *WXUNUSED(widget),
 }
 }
 
+//-----------------------------------------------------------------------------
+// emission hook for "parent-set"
+//-----------------------------------------------------------------------------
+
+extern "C" {
+static gboolean
+parent_set_hook(GSignalInvocationHint*, guint, const GValue* param_values, void* data)
+{
+    wxGLCanvas* win = (wxGLCanvas*)data;
+    if (g_value_peek_pointer(&param_values[0]) == win->m_wxwindow)
+    {
+        const XVisualInfo* xvi = win->GetXVisualInfo();
+        GdkVisual* visual = gtk_widget_get_visual(win->m_wxwindow);
+        if (GDK_VISUAL_XVISUAL(visual)->visualid != xvi->visualid)
+        {
+            GdkScreen* screen = gtk_widget_get_screen(win->m_wxwindow);
+            visual = gdk_x11_screen_lookup_visual(screen, xvi->visualid);
+            GdkColormap* colormap = gdk_colormap_new(visual, false);
+            gtk_widget_set_colormap(win->m_wxwindow, colormap);
+            g_object_unref(colormap);
+        }
+        // remove hook
+        return false;
+    }
+    return true;
+}
+}
+
 //---------------------------------------------------------------------------
 // wxGlCanvas
 //---------------------------------------------------------------------------
@@ -186,27 +214,16 @@ bool wxGLCanvas::Create(wxWindow *parent,
     if ( !InitVisual(attribList) )
         return false;
 
-    XVisualInfo * const xvi = GetXVisualInfo();
-
-    GdkVisual *visual;
-    GdkColormap *colormap;
+    // watch for the "parent-set" signal on m_wxwindow so we can set colormap
+    // before m_wxwindow is realized (which will occur before
+    // wxWindow::Create() returns if parent is already visible)
+    unsigned sig_id = g_signal_lookup("parent-set", GTK_TYPE_WIDGET);
+    g_signal_add_emission_hook(sig_id, 0, parent_set_hook, this, NULL);
 
     wxWindow::Create( parent, id, pos, size, style, name );
 
     m_glWidget = m_wxwindow;
 
-    GdkScreen *screen = gtk_widget_get_screen( m_glWidget );
-    colormap = gdk_screen_get_default_colormap(screen);
-    visual = gdk_colormap_get_visual(colormap);
-
-    if (GDK_VISUAL_XVISUAL(visual)->visualid != xvi->visualid)
-    {
-        visual = gdk_x11_screen_lookup_visual( screen, xvi->visualid );
-        colormap = gdk_colormap_new(visual, FALSE);
-    }
-
-    gtk_widget_set_colormap( m_glWidget, colormap );
-
     gtk_widget_set_double_buffered( m_glWidget, FALSE );
 
 #if WXWIN_COMPATIBILITY_2_8