Simplify transactions by utilising a dirty flag on containers

This PR changes the way we handle transactions to a more simple method.
The new method is to mark containers as dirty from low level code
(eg. arranging, or container_destroy, and eventually seat_set_focus),
then call transaction_commit_dirty which picks up those containers and
runs them through a transaction. The old methods of using transactions
(arrange_and_commit, or creating one manually) are now no longer
possible.

The highest-level code (execute_command and view implementation
handlers) will call transaction_commit_dirty, so most other code just
needs to set containers as dirty. This is done by arranging, but can
also be done by calling container_set_dirty.

1 files changed, 2 insertions, 3 deletions

diff --git a/sway/server.c b/sway/server.c
index 1d8eb964..8566d512 100644
--- a/sway/server.c
+++ b/sway/server.c
@@ -125,8 +125,7 @@ bool server_init(struct sway_server *server) {
         if (debug != NULL && strcmp(debug, "txn_timings") == 0) {
                 server->debug_txn_timings = true;
         }
-        server->destroying_containers = create_list();
-
+        server->dirty_containers = create_list();
         server->transactions = create_list();
 
         input_manager = input_manager_create(server);
@@ -136,7 +135,7 @@ bool server_init(struct sway_server *server) {
 void server_fini(struct sway_server *server) {
         // TODO: free sway-specific resources
         wl_display_destroy(server->wl_display);
-        list_free(server->destroying_containers);
+        list_free(server->dirty_containers);
         list_free(server->transactions);
 }