wayland: Move warning about transient-less temporaries to map()

We shouldn't warn in a funtion that seemingly only checks how something
eventually should be mapped.

Closes: https://gitlab.gnome.org/GNOME/gtk/-/issues/2589
This commit is contained in:
Jonas Ådahl
2020-04-08 11:40:10 +02:00
parent 99bae0fb5f
commit cf060b4c7d

View File

@@ -2955,10 +2955,6 @@ should_map_as_popup (GdkWindow *window)
if (impl->grab_input_seat)
return TRUE;
}
else
g_message ("Window %p is a temporary window without parent, "
"application will not be able to position it on screen.",
window);
}
/* Yet we need to keep the window type hint tests for compatibility */
@@ -3043,6 +3039,16 @@ gdk_wayland_window_map (GdkWindow *window)
if (impl->mapped || impl->use_custom_surface)
return;
if (GDK_WINDOW_TYPE (window) == GDK_WINDOW_TEMP)
{
if (!impl->transient_for)
{
g_message ("Window %p is a temporary window without parent, "
"application will not be able to position it on screen.",
window);
}
}
if (should_map_as_subsurface (window))
{
if (impl->transient_for)