transaction: validate X transaction completions by geometry, not size

Xwayland views are aware of their coordinates, so validating transaction
completions should take into account the reported coordinates of the
view. Prior to this commit they didn't, and matching dimensions would
suffice to validate the transaction.

Also introduced `transaction_notify_view_ready_immediately` to support
the fix from d0f7e0f without jumping through hoops to figure out the
geometry of an `xdg_shell` view.

1 files changed, 4 insertions, 4 deletions

diff --git a/sway/desktop/xwayland.c b/sway/desktop/xwayland.c
index cee0ab10..186502b2 100644
--- a/sway/desktop/xwayland.c
+++ b/sway/desktop/xwayland.c
@@ -401,8 +401,8 @@ static void handle_commit(struct wl_listener *listener, void *data) {
 
         if (view->container->node.instruction) {
                 get_geometry(view, &view->geometry);
-                transaction_notify_view_ready_by_size(view,
-                                state->width, state->height);
+                transaction_notify_view_ready_by_geometry(view,
+                                xsurface->x, xsurface->y, state->width, state->height);
         } else {
                 struct wlr_box new_geo;
                 get_geometry(view, &new_geo);
@@ -418,8 +418,8 @@ static void handle_commit(struct wl_listener *listener, void *data) {
                         memcpy(&view->geometry, &new_geo, sizeof(struct wlr_box));
                         desktop_damage_view(view);
                         transaction_commit_dirty();
-                        transaction_notify_view_ready_by_size(view,
-                                        new_geo.width, new_geo.height);
+                        transaction_notify_view_ready_by_geometry(view,
+                                        xsurface->x, xsurface->y, new_geo.width, new_geo.height);
                 } else {
                         memcpy(&view->geometry, &new_geo, sizeof(struct wlr_box));
                 }