4 files changed, 90 insertions, 53 deletions

diff --git a/sway/desktop/launcher.c b/sway/desktop/launcher.c
index 28043d19..2362e1ba 100644
--- a/sway/desktop/launcher.c
+++ b/sway/desktop/launcher.c
@@ -4,6 +4,7 @@
 #include "sway/input/seat.h"
 #include "sway/output.h"
 #include "sway/desktop/launcher.h"
+#include "sway/server.h"
 #include "sway/tree/node.h"
 #include "sway/tree/container.h"
 #include "sway/tree/workspace.h"
diff --git a/sway/desktop/layer_shell.c b/sway/desktop/layer_shell.c
index 4b2584b6..6221b7b9 100644
--- a/sway/desktop/layer_shell.c
+++ b/sway/desktop/layer_shell.c
@@ -2,6 +2,7 @@
 #include <stdlib.h>
 #include <string.h>
 #include <wayland-server-core.h>
+#include <wlr/types/wlr_fractional_scale_v1.h>
 #include <wlr/types/wlr_layer_shell_v1.h>
 #include <wlr/types/wlr_output.h>
 #include <wlr/types/wlr_scene.h>
@@ -432,6 +433,12 @@ void handle_layer_shell_surface(struct wl_listener *listener, void *data) {
 
         surface->output = output;
 
+        // now that the surface's output is known, we can advertise its scale
+        wlr_fractional_scale_v1_notify_scale(surface->layer_surface->surface,
+                layer_surface->output->scale);
+        wlr_surface_set_preferred_buffer_scale(surface->layer_surface->surface,
+                ceil(layer_surface->output->scale));
+
         surface->surface_commit.notify = handle_surface_commit;
         wl_signal_add(&layer_surface->surface->events.commit,
                 &surface->surface_commit);
diff --git a/sway/desktop/output.c b/sway/desktop/output.c
index b8f2d32d..6bf77d17 100644
--- a/sway/desktop/output.c
+++ b/sway/desktop/output.c
@@ -243,10 +243,14 @@ static int output_repaint_timer_handler(void *data) {
 
         output_configure_scene(output, &root->root_scene->tree.node, 1.0f);
 
+        struct wlr_scene_output_state_options opts = {
+                .color_transform = output->color_transform,
+        };
+
         if (output->gamma_lut_changed) {
                 struct wlr_output_state pending;
                 wlr_output_state_init(&pending);
-                if (!wlr_scene_output_build_state(output->scene_output, &pending, NULL)) {
+                if (!wlr_scene_output_build_state(output->scene_output, &pending, &opts)) {
                         return 0;
                 }
 
@@ -269,7 +273,7 @@ static int output_repaint_timer_handler(void *data) {
                 return 0;
         }
 
-        wlr_scene_output_commit(output->scene_output, NULL);
+        wlr_scene_output_commit(output->scene_output, &opts);
         return 0;
 }
 
@@ -521,9 +525,7 @@ void handle_new_output(struct wl_listener *listener, void *data) {
                 sway_session_lock_add_output(server->session_lock.lock, output);
         }
 
-        struct output_config *oc = find_output_config(output);
-        apply_output_config(oc, output);
-        free_output_config(oc);
+        apply_all_output_configs();
 
         transaction_commit_dirty();
 
@@ -552,63 +554,89 @@ void handle_gamma_control_set_gamma(struct wl_listener *listener, void *data) {
         wlr_output_schedule_frame(output->wlr_output);
 }
 
+static struct output_config *output_config_for_config_head(
+                struct wlr_output_configuration_head_v1 *config_head,
+                struct sway_output *output) {
+        struct output_config *oc = new_output_config(output->wlr_output->name);
+        oc->enabled = config_head->state.enabled;
+        if (!oc->enabled) {
+                return oc;
+        }
+
+        if (config_head->state.mode != NULL) {
+                struct wlr_output_mode *mode = config_head->state.mode;
+                oc->width = mode->width;
+                oc->height = mode->height;
+                oc->refresh_rate = mode->refresh / 1000.f;
+        } else {
+                oc->width = config_head->state.custom_mode.width;
+                oc->height = config_head->state.custom_mode.height;
+                oc->refresh_rate =
+                        config_head->state.custom_mode.refresh / 1000.f;
+        }
+        oc->x = config_head->state.x;
+        oc->y = config_head->state.y;
+        oc->transform = config_head->state.transform;
+        oc->scale = config_head->state.scale;
+        oc->adaptive_sync = config_head->state.adaptive_sync_enabled;
+        return oc;
+}
+
 static void output_manager_apply(struct sway_server *server,
                 struct wlr_output_configuration_v1 *config, bool test_only) {
-        // TODO: perform atomic tests on the whole backend atomically
-
-        struct wlr_output_configuration_head_v1 *config_head;
-        // First disable outputs we need to disable
-        bool ok = true;
-        wl_list_for_each(config_head, &config->heads, link) {
-                struct wlr_output *wlr_output = config_head->state.output;
-                struct sway_output *output = wlr_output->data;
-                if (!output->enabled || config_head->state.enabled) {
+        size_t configs_len = wl_list_length(&root->all_outputs);
+        struct matched_output_config *configs = calloc(configs_len, sizeof(*configs));
+        if (!configs) {
+                return;
+        }
+
+        int config_idx = 0;
+        struct sway_output *sway_output;
+        wl_list_for_each(sway_output, &root->all_outputs, link) {
+                if (sway_output == root->fallback_output) {
+                        configs_len--;
                         continue;
                 }
-                struct output_config *oc = new_output_config(output->wlr_output->name);
-                oc->enabled = false;
 
-                if (test_only) {
-                        ok &= test_output_config(oc, output);
-                } else {
-                        oc = store_output_config(oc);
-                        ok &= apply_output_config(oc, output);
+                struct matched_output_config *cfg = &configs[config_idx++];
+                cfg->output = sway_output;
+
+                struct wlr_output_configuration_head_v1 *config_head;
+                wl_list_for_each(config_head, &config->heads, link) {
+                        if (config_head->state.output == sway_output->wlr_output) {
+                                cfg->config = output_config_for_config_head(config_head, sway_output);
+                                break;
+                        }
+                }
+                if (!cfg->config) {
+                        cfg->config = find_output_config(sway_output);
                 }
         }
 
-        // Then enable outputs that need to
-        wl_list_for_each(config_head, &config->heads, link) {
-                struct wlr_output *wlr_output = config_head->state.output;
-                struct sway_output *output = wlr_output->data;
-                if (!config_head->state.enabled) {
-                        continue;
-                }
-                struct output_config *oc = new_output_config(output->wlr_output->name);
-                oc->enabled = true;
-                if (config_head->state.mode != NULL) {
-                        struct wlr_output_mode *mode = config_head->state.mode;
-                        oc->width = mode->width;
-                        oc->height = mode->height;
-                        oc->refresh_rate = mode->refresh / 1000.f;
-                } else {
-                        oc->width = config_head->state.custom_mode.width;
-                        oc->height = config_head->state.custom_mode.height;
-                        oc->refresh_rate =
-                                config_head->state.custom_mode.refresh / 1000.f;
+        sort_output_configs_by_priority(configs, configs_len);
+        bool ok = apply_output_configs(configs, configs_len, test_only, false);
+        for (size_t idx = 0; idx < configs_len; idx++) {
+                struct matched_output_config *cfg = &configs[idx];
+
+                // Only store new configs for successful non-test commits. Old configs,
+                // test-only and failed commits just get freed.
+                bool store_config = false;
+                if (!test_only && ok) {
+                        struct wlr_output_configuration_head_v1 *config_head;
+                        wl_list_for_each(config_head, &config->heads, link) {
+                                if (config_head->state.output == cfg->output->wlr_output) {
+                                        store_config = true;
+                                        break;
+                                }
+                        }
                 }
-                oc->x = config_head->state.x;
-                oc->y = config_head->state.y;
-                oc->transform = config_head->state.transform;
-                oc->scale = config_head->state.scale;
-                oc->adaptive_sync = config_head->state.adaptive_sync_enabled;
-
-                if (test_only) {
-                        ok &= test_output_config(oc, output);
+                if (store_config) {
+                        store_output_config(cfg->config);
                 } else {
-                        oc = store_output_config(oc);
-                        ok &= apply_output_config(oc, output);
+                        free_output_config(cfg->config);
                 }
         }
+        free(configs);
 
         if (ok) {
                 wlr_output_configuration_v1_send_succeeded(config);
@@ -652,6 +680,6 @@ void handle_output_power_manager_set_mode(struct wl_listener *listener,
                 oc->power = 1;
                 break;
         }
-        oc = store_output_config(oc);
-        apply_output_config(oc, output);
+        store_output_config(oc);
+        apply_all_output_configs();
 }
diff --git a/sway/desktop/transaction.c b/sway/desktop/transaction.c
index 042141ab..d1898843 100644
--- a/sway/desktop/transaction.c
+++ b/sway/desktop/transaction.c
@@ -10,6 +10,7 @@
 #include "sway/input/cursor.h"
 #include "sway/input/input-manager.h"
 #include "sway/output.h"
+#include "sway/server.h"
 #include "sway/tree/container.h"
 #include "sway/tree/node.h"
 #include "sway/tree/view.h"
@@ -761,7 +762,7 @@ static bool should_configure(struct sway_node *node,
         }
         struct sway_container_state *cstate = &node->sway_container->current;
         struct sway_container_state *istate = &instruction->container_state;
-#if HAVE_XWAYLAND
+#if WLR_HAS_XWAYLAND
         // Xwayland views are position-aware and need to be reconfigured
         // when their position changes.
         if (node->sway_container->view->type == SWAY_VIEW_XWAYLAND) {