tiling: Treat a maximized window as tiled for the purposes of

determining whether or not to save the window's original position
and size.

Windows that are maximized via tiling shortcuts are META_TILE_MAXIMIZED,
but those maximized using other methods were getting missed and
their original save position was being overwritten when moving
from maximized to tiled via shortcut.

ref: linuxmint/mint21-beta#75

Author: mtwebster

src/core/keybindings.c

@@ -3510,14 +3510,6 @@ handle_tile_action (MetaDisplay     *display,
   else if (can_tile (window, new_mode))
     {
       window->tile_monitor_number = window->monitor->number;
-      /* Maximization constraints beat tiling constraints, so if the window
-       * is maximized, tiling won't have any effect unless we unmaximize it
-       * horizontally first; rather than calling meta_window_unmaximize(),
-       * we just set the flag and rely on meta_window_tile() syncing it to
-       * save an additional roundtrip.
-       */
-      window->maximized_horizontally = FALSE;
-      window->maximized_vertically = FALSE;
       meta_window_tile (window, new_mode);
     }
 }

src/core/window.c

@@ -3403,7 +3403,15 @@ meta_window_tile (MetaWindow   *window,
 {
   MetaMaximizeFlags directions;
   MetaRectangle old_frame_rect, old_buffer_rect;
-  gboolean was_already_tiled = window->tile_mode != META_TILE_NONE;
+  gboolean was_already_tiled = window->tile_mode != META_TILE_NONE || META_WINDOW_MAXIMIZED (window);
+  /* Maximization constraints beat tiling constraints, so if the window
+   * is maximized, tiling won't have any effect unless we unmaximize it
+   * horizontally first; rather than calling meta_window_unmaximize(),
+   * we just set the flag and rely on meta_window_tile() syncing it to
+   * save an additional roundtrip.
+   */
+  window->maximized_horizontally = FALSE;
+  window->maximized_vertically = FALSE;
 
   meta_window_get_tile_fractions (window, tile_mode, &window->tile_hfraction, &window->tile_vfraction);
   window->tile_mode = tile_mode;