[mutter] wayland: Apply size hints regardless of geometry

[Olivier Fourdan <ofourdan src gnome org>]

commit f241bdba934f938c0a97587806d02c0f2464f221
Author: Olivier Fourdan <ofourdan redhat com>
Date:   Wed May 10 08:59:53 2017 +0200

    wayland: Apply size hints regardless of geometry
    
    Previously we would bail out early in xdg_toplevel_role_commit() if no
    geometry change was set, ignoring the possible min/max size hints
    changes.
    
    But setting a min/max size hint without changing the geometry is
    perfectly valid, so we ought to apply the min/max changes regardless of
    a geometry change.
    
    https://bugzilla.gnome.org/show_bug.cgi?id=782213

 src/wayland/meta-wayland-xdg-shell.c |   22 ++++++++++------------
 1 files changed, 10 insertions(+), 12 deletions(-)
---
diff --git a/src/wayland/meta-wayland-xdg-shell.c b/src/wayland/meta-wayland-xdg-shell.c
index ef9a2f5..88e0cae 100644
--- a/src/wayland/meta-wayland-xdg-shell.c
+++ b/src/wayland/meta-wayland-xdg-shell.c
@@ -624,20 +624,18 @@ xdg_toplevel_role_commit (MetaWaylandSurfaceRole  *surface_role,
   if (!window)
     return;
 
-  if (!pending->has_new_geometry)
+  if (pending->has_new_geometry)
     {
-      if (pending->dx != 0 || pending->dx != 0)
-        {
-          g_warning ("XXX: Attach-initiated move without a new geometry. This is unimplemented right now.");
-        }
-      return;
+      window_geometry = meta_wayland_xdg_surface_get_window_geometry (xdg_surface);
+      meta_window_wayland_move_resize (window,
+                                       &xdg_surface_priv->acked_configure_serial,
+                                       window_geometry,
+                                       pending->dx, pending->dy);
+    }
+  else if (pending->dx != 0 || pending->dx != 0)
+    {
+      g_warning ("XXX: Attach-initiated move without a new geometry. This is unimplemented right now.");
     }
-
-  window_geometry = meta_wayland_xdg_surface_get_window_geometry (xdg_surface);
-  meta_window_wayland_move_resize (window,
-                                   &xdg_surface_priv->acked_configure_serial,
-                                   window_geometry,
-                                   pending->dx, pending->dy);
 
   /* When we get to this point, we ought to have valid size hints */
   if (pending->has_new_min_size || pending->has_new_max_size)