[895/906] context: call window_class->close last
authorMatthew Waters <ystreet00@gmail.com>
Fri, 28 Feb 2014 06:42:51 +0000 (17:42 +1100)
committerTim-Philipp Müller <tim@centricular.com>
Sat, 9 Dec 2017 19:31:33 +0000 (19:31 +0000)
We should destroy resources before closing the display connection

Fixes https://bugzilla.gnome.org/show_bug.cgi?id=725048

gst-libs/gst/gl/gstglcontext.c

index 1051d3b..f55c13f 100644 (file)
@@ -792,17 +792,19 @@ gst_gl_context_create_thread (GstGLContext * context)
 
   context->priv->alive = FALSE;
 
-  if (window_class->close) {
-    window_class->close (context->window);
-  }
-
   context_class->activate (context, FALSE);
 
   context_class->destroy_context (context);
 
+  /* User supplied callback */
   if (context->window->close)
     context->window->close (context->window->close_data);
 
+  /* window specific shutdown */
+  if (window_class->close) {
+    window_class->close (context->window);
+  }
+
   g_cond_signal (&context->priv->destroy_cond);
 
   g_mutex_unlock (&context->priv->render_lock);