Changeset 6828a56 in mainline

Timestamp:

2023-01-19T13:51:14Z (15 months ago)

Author:

Jiri Svoboda <jiri@…>

Branches:

master, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export

Children:

5d380b6

Parents:

3be5366

git-author:

Jiri Svoboda <jiri@…> (2023-01-19 19:50:50)

git-committer:

Jiri Svoboda <jiri@…> (2023-01-19 13:51:14)

Message:

Avoid two seats fighting when moving/resizing a window

When two seats have focus on the same window, both pointer's movements
would be delivered to it. Make sure only the seat which started the
move or resize operation affects the operation, otherwise they fight
over it.

Location:

uspace/srv/hid/display

Files:

• test/window.c (modified) (2 diffs)
• window.c (modified) (3 diffs)
• window.h (modified) (1 diff)

uspace/srv/hid/display/test/window.c

@@ -36 +36 @@
 #include "../client.h"
 #include "../display.h"
+#include "../idevcfg.h"
 #include "../seat.h"
 #include "../window.h"
@@ -1273 +1274 @@
 }
 
+/** ds_window_orig_seat() correctly compares seats */
+PCUT_TEST(window_orig_seat)
+{
+        gfx_context_t *gc;
+        ds_display_t *disp;
+        ds_client_t *client;
+        ds_seat_t *seat0;
+        ds_seat_t *seat1;
+        sysarg_t devid0;
+        sysarg_t devid1;
+        ds_idevcfg_t *cfg0;
+        ds_idevcfg_t *cfg1;
+        ds_window_t *wnd;
+        display_wnd_params_t params;
+        errno_t rc;
+
+        rc = gfx_context_new(&dummy_ops, NULL, &gc);
+        PCUT_ASSERT_ERRNO_VAL(EOK, rc);
+
+        rc = ds_display_create(gc, df_none, &disp);
+        PCUT_ASSERT_ERRNO_VAL(EOK, rc);
+
+        rc = ds_client_create(disp, NULL, NULL, &client);
+        PCUT_ASSERT_ERRNO_VAL(EOK, rc);
+
+        rc = ds_seat_create(disp, "Alice", &seat0);
+        PCUT_ASSERT_ERRNO_VAL(EOK, rc);
+
+        rc = ds_seat_create(disp, "Bob", &seat1);
+        PCUT_ASSERT_ERRNO_VAL(EOK, rc);
+
+        display_wnd_params_init(&params);
+        params.rect.p0.x = params.rect.p0.y = 0;
+        params.rect.p1.x = params.rect.p1.y = 1;
+
+        rc = ds_window_create(client, &params, &wnd);
+        PCUT_ASSERT_ERRNO_VAL(EOK, rc);
+
+        devid0 = 42;
+        devid1 = 43;
+
+        rc = ds_idevcfg_create(disp, devid0, seat0, &cfg0);
+        PCUT_ASSERT_ERRNO_VAL(EOK, rc);
+
+        rc = ds_idevcfg_create(disp, devid1, seat1, &cfg1);
+        PCUT_ASSERT_ERRNO_VAL(EOK, rc);
+
+        wnd->state = dsw_moving;
+        wnd->orig_pos_id = devid0;
+
+        PCUT_ASSERT_TRUE(ds_window_orig_seat(wnd, devid0));
+        PCUT_ASSERT_FALSE(ds_window_orig_seat(wnd, devid1));
+
+        ds_idevcfg_destroy(cfg0);
+        ds_idevcfg_destroy(cfg1);
+        ds_window_destroy(wnd);
+        ds_seat_destroy(seat0);
+        ds_seat_destroy(seat1);
+        ds_client_destroy(client);
+        ds_display_destroy(disp);
+}
+
 static errno_t dummy_set_color(void *arg, gfx_color_t *color)
 {

uspace/srv/hid/display/window.c

@@ -656 +656 @@
 
         if (event->type == POS_RELEASE) {
-                if (wnd->state == dsw_moving) {
+                /* Finish move/resize if they were started by the same seat */
+                if (wnd->state == dsw_moving &&
+                    ds_window_orig_seat(wnd, pos_id)) {
                         ds_window_finish_move(wnd, &pos);
                         return EOK;
                 }
 
-                if (wnd->state == dsw_resizing) {
+                if (wnd->state == dsw_resizing &&
+                    ds_window_orig_seat(wnd, pos_id)) {
                         ds_window_finish_resize(wnd, &pos);
                         return EOK;
@@ -668 +671 @@
 
         if (event->type == POS_UPDATE) {
-                if (wnd->state == dsw_moving) {
+                /* Update move/resize if they were started by the same seat */
+                if (wnd->state == dsw_moving &&
+                    ds_window_orig_seat(wnd, pos_id)) {
                         ds_window_update_move(wnd, &pos);
                         return EOK;
                 }
 
-                if (wnd->state == dsw_resizing) {
+                if (wnd->state == dsw_resizing &&
+                    ds_window_orig_seat(wnd, pos_id)) {
                         ds_window_update_resize(wnd, &pos);
                         return EOK;
@@ -1125 +1131 @@
 }
 
+/** Determine if input device belongs to the same seat as the original device.
+ *
+ * Compare the seat ownning @a idev_id with the seat owning @a wnd->orig_pos_id
+ * (the device that started the window move or resize).
+ *
+ * This is used to make sure that, when two seats focus the same window,
+ * only devices owned by the seat that started the resize or move can
+ * affect it. Otherwise moving the other pointer(s) would disrupt the
+ * resize or move operation.
+ *
+ * @param wnd Window (that is currently being resized or moved)
+ * @param idev_id Input device ID
+ * @return @c true iff idev_id is owned by the same seat as the input
+ *         device that started the resize or move
+ */
+bool ds_window_orig_seat(ds_window_t *wnd, sysarg_t idev_id)
+{
+        ds_seat_t *orig_seat;
+        ds_seat_t *seat;
+
+        /* Window must be in state such that wnd->orig_pos_id is valid */
+        assert(wnd->state == dsw_moving || wnd->state == dsw_resizing);
+
+        orig_seat = ds_display_seat_by_idev(wnd->display, wnd->orig_pos_id);
+        seat = ds_display_seat_by_idev(wnd->display, idev_id);
+
+        return seat == orig_seat;
+}
+
 /** Window memory GC invalidate callback.
  *

uspace/srv/hid/display/window.h

@@ -80 +80 @@
 extern ds_window_t *ds_window_find_alt(ds_window_t *, display_wnd_flags_t);
 extern void ds_window_unfocus(ds_window_t *);
+extern bool ds_window_orig_seat(ds_window_t *, sysarg_t);
 
 #endif