Introduce seat_set_raw_focus and remove notify argument from seat_set_focus_warp

This introduces seat_set_raw_focus: a function that manipulates the
focus stack without doing any other behaviour whatsoever. There are a
few places where this is useful, such as where we set focus_inactive
followed by another call to set the real focus again. With this change,
the notify argument to seat_set_focus_warp is also removed as these
cases now use the raw function instead.

A bonus of this is we are no longer emitting window::focus IPC events
when setting focus_inactive, nor are we sending focus/unfocus events to
the surface.

This also fixes the following:

* When running `move workspace to output <name>` and moving the last
workspace from the source output, the workspace::focus IPC event is no
longer emitted for the newly created workspace.
* When splitting the currently focused container, unfocus/focus events
will not be sent to the surface when giving focus_inactive to the newly
created parent, and window::focus events will not be emitted.

1 files changed, 3 insertions, 3 deletions

diff --git a/sway/commands/move.c b/sway/commands/move.c
index fc2f1cc1..215ffe27 100644
--- a/sway/commands/move.c
+++ b/sway/commands/move.c
@@ -543,7 +543,7 @@ static struct cmd_results *cmd_move_container(int argc, char **argv) {
         if (new_output_last_ws && new_output_last_ws != new_workspace) {
                 struct sway_node *new_output_last_focus =
                         seat_get_focus_inactive(seat, &new_output_last_ws->node);
-                seat_set_focus_warp(seat, new_output_last_focus, false, false);
+                seat_set_raw_focus(seat, new_output_last_focus);
         }
 
         // restore focus
@@ -556,7 +556,7 @@ static struct cmd_results *cmd_move_container(int argc, char **argv) {
                         focus = seat_get_focus_inactive(seat, &old_ws->node);
                 }
         }
-        seat_set_focus_warp(seat, focus, true, false);
+        seat_set_focus(seat, focus);
 
         // clean-up, destroying parents if the container was the last child
         if (old_parent) {
@@ -593,7 +593,7 @@ static void workspace_move_to_output(struct sway_workspace *workspace,
                 char *ws_name = workspace_next_name(old_output->wlr_output->name);
                 struct sway_workspace *ws = workspace_create(old_output, ws_name);
                 free(ws_name);
-                seat_set_focus_workspace(seat, ws);
+                seat_set_raw_focus(seat, &ws->node);
         }
 
         workspace_consider_destroy(new_output_old_ws);