Revert "client: Don't move clients around across restarts (FS#1159)"

This reverts commit c3bca6ac34.

This caused problems where clients would always spawn on the top-left screen and
couldn't be moved around. This was because lua placed them on the other screen,
but when the titlebar got added, their top-left corner was moved over again.

awesome.c

@@ -65,12 +65,6 @@ awesome_atexit(bool restart)
     lua_pushboolean(globalconf.L, restart);
     signal_object_emit(globalconf.L, &global_signals, "exit", 1);
 
-    /* TODO: Reparent windows back to root window, placing them according to
-     * their window gravity. Right now we pretend to use static gravity and that
-     * works automatically thanks to xcb_change_save_set(). See
-     * titlebar_resize().
-     */
-
     a_dbus_cleanup();
 
     systray_cleanup();

objects/client.c

@@ -1558,23 +1558,16 @@ titlebar_resize(client_t *c, client_titlebar_t bar, int size)
         return;
 
     /* Now resize the client (and titlebars!) suitably (the client without
-     * titlebars should keep its current size!)
+     * titlebars should keep its current size!) */
-     * TODO: I guess that this should honor the client's window gravity, right
-     * now we pretend to have static gravity. See awesome_atexit().
-     */
     area_t geometry = c->geometry;
     int change = size - c->titlebar[bar].size;
     switch (bar) {
     case CLIENT_TITLEBAR_TOP:
-        geometry.y -= change;
-        /* fall through */;
     case CLIENT_TITLEBAR_BOTTOM:
         geometry.height += change;
         break;
-    case CLIENT_TITLEBAR_LEFT:
-        geometry.x -= change;
-        /* fall through */;
     case CLIENT_TITLEBAR_RIGHT:
+    case CLIENT_TITLEBAR_LEFT:
         geometry.width += change;
         break;
     default: