diff options
| -rw-r--r-- | include/wlr/types/wlr_layer_shell.h | 2 | ||||
| -rw-r--r-- | protocol/wlr-layer-shell-unstable-v1.xml | 10 | ||||
| -rw-r--r-- | types/wlr_layer_shell.c | 65 |
3 files changed, 56 insertions, 21 deletions
diff --git a/include/wlr/types/wlr_layer_shell.h b/include/wlr/types/wlr_layer_shell.h index 33bda351..58c12459 100644 --- a/include/wlr/types/wlr_layer_shell.h +++ b/include/wlr/types/wlr_layer_shell.h @@ -40,7 +40,7 @@ struct wlr_layer_surface_state { struct wlr_layer_surface_configure { struct wl_list link; // wlr_layer_surface::configure_list uint32_t serial; - struct wlr_layer_surface_state *state; + struct wlr_layer_surface_state state; }; struct wlr_layer_surface { diff --git a/protocol/wlr-layer-shell-unstable-v1.xml b/protocol/wlr-layer-shell-unstable-v1.xml index 494b29ca..29ba9dd4 100644 --- a/protocol/wlr-layer-shell-unstable-v1.xml +++ b/protocol/wlr-layer-shell-unstable-v1.xml @@ -131,10 +131,10 @@ Margin is double-buffered, see wl_surface.commit. </description> - <arg name="top" type="int"/> - <arg name="right" type="int"/> - <arg name="bottom" type="int"/> - <arg name="left" type="int"/> + <arg name="top" type="uint"/> + <arg name="right" type="uint"/> + <arg name="bottom" type="uint"/> + <arg name="left" type="uint"/> </request> <request name="get_popup"> @@ -217,6 +217,8 @@ <enum name="error"> <entry name="invalid_input_device" value="0" summary="input device bitfield is invalid"/> <entry name="invalid_anchor" value="1" summary="anchor bitfield is invalid"/> + <entry name="invalid_surface_state" value="2" + summary="the client provided an invalid surface state"/> </enum> <enum name="anchor" bitfield="true"> diff --git a/types/wlr_layer_shell.c b/types/wlr_layer_shell.c index 120f2197..055578bd 100644 --- a/types/wlr_layer_shell.c +++ b/types/wlr_layer_shell.c @@ -31,25 +31,68 @@ static struct wlr_layer_surface *layer_surface_from_resource( return wl_resource_get_user_data(resource); } +static void layer_surface_configure_destroy( + struct wlr_layer_surface_configure *configure) { + if (configure == NULL) { + return; + } + wl_list_remove(&configure->link); + free(configure); +} + static void layer_surface_handle_ack_configure(struct wl_client *client, struct wl_resource *resource, uint32_t serial) { - // TODO + struct wlr_layer_surface *surface = layer_surface_from_resource(resource); + + bool found = false; + struct wlr_layer_surface_configure *configure, *tmp; + wl_list_for_each_safe(configure, tmp, &surface->configure_list, link) { + if (configure->serial < serial) { + layer_surface_configure_destroy(configure); + } else if (configure->serial == serial) { + found = true; + break; + } else { + break; + } + } + if (!found) { + wl_resource_post_error(surface->client->resource, + ZWLR_LAYER_SURFACE_V1_ERROR_INVALID_SURFACE_STATE, + "wrong configure serial: %u", serial); + return; + } + + surface->configured = true; + surface->configure_serial = serial; + + surface->current.anchor = configure->state.anchor; + surface->current.exclusive_zone = configure->state.exclusive_zone; + surface->current.margin = configure->state.margin; + + layer_surface_configure_destroy(configure); } static void layer_surface_handle_set_anchor(struct wl_client *client, struct wl_resource *resource, uint32_t anchor) { - // TODO + struct wlr_layer_surface *surface = layer_surface_from_resource(resource); + surface->next.anchor = anchor; } static void layer_surface_handle_set_exclusive_zone(struct wl_client *client, struct wl_resource *resource, uint32_t zone) { - // TODO + struct wlr_layer_surface *surface = layer_surface_from_resource(resource); + surface->next.exclusive_zone = zone; } static void layer_surface_handle_set_margin(struct wl_client *client, - struct wl_resource *resource, int32_t top, int32_t right, - int32_t bottom, int32_t left) { - // TODO + struct wl_resource *resource, uint32_t top, + uint32_t right, uint32_t bottom, uint32_t left) { + struct wlr_layer_surface *surface = layer_surface_from_resource(resource); + surface->next.margin.top = top; + surface->next.margin.right = right; + surface->next.margin.bottom = bottom; + surface->next.margin.left = left; } static void layer_surface_handle_get_popup(struct wl_client *client, @@ -72,16 +115,6 @@ static const struct zwlr_layer_surface_v1_interface layer_surface_implementation .get_input = layer_surface_handle_get_input, }; -static void layer_surface_configure_destroy( - struct wlr_layer_surface_configure *configure) { - if (configure == NULL) { - return; - } - wl_list_remove(&configure->link); - free(configure->state); - free(configure); -} - static void layer_surface_unmap(struct wlr_layer_surface *surface) { // TODO: probably need to ungrab before this event wlr_signal_emit_safe(&surface->events.unmap, surface); |