[ShellApp] Move assertion below precondition check

Need to track down why we're calling _remove_window with a window not
in the app, but this is the quick fix for a crasher.
This commit is contained in:
Colin Walters 2010-06-19 11:32:00 -04:00
parent e63a0f3a13
commit 1f550dbc72

View File

@ -757,11 +757,12 @@ void
_shell_app_remove_window (ShellApp *app, _shell_app_remove_window (ShellApp *app,
MetaWindow *window) MetaWindow *window)
{ {
g_assert (app->running_state != NULL);
if (!g_slist_find (app->running_state->windows, window)) if (!g_slist_find (app->running_state->windows, window))
return; return;
g_assert (app->running_state != NULL);
g_signal_handlers_disconnect_by_func (window, G_CALLBACK(shell_app_on_unmanaged), app); g_signal_handlers_disconnect_by_func (window, G_CALLBACK(shell_app_on_unmanaged), app);
g_signal_handlers_disconnect_by_func (window, G_CALLBACK(shell_app_on_user_time_changed), app); g_signal_handlers_disconnect_by_func (window, G_CALLBACK(shell_app_on_user_time_changed), app);
g_object_unref (window); g_object_unref (window);