Fix ack configure/configure flow

Prevents FOUC/improves frame perfect rendering goal
This commit is contained in:
Drew DeVault 2018-03-27 22:50:55 -04:00
parent 3e5131e664
commit 96a8df2f9a
2 changed files with 31 additions and 16 deletions

View file

@ -64,6 +64,8 @@ struct wlr_layer_surface {
uint32_t configure_next_serial; uint32_t configure_next_serial;
struct wl_list configure_list; struct wl_list configure_list;
struct wlr_layer_surface_configure *acked_configure;
struct wlr_layer_surface_state client_pending; struct wlr_layer_surface_state client_pending;
struct wlr_layer_surface_state server_pending; struct wlr_layer_surface_state server_pending;
struct wlr_layer_surface_state current; struct wlr_layer_surface_state current;

View file

@ -64,12 +64,12 @@ static void layer_surface_handle_ack_configure(struct wl_client *client,
return; return;
} }
surface->configured = true; if (surface->acked_configure) {
surface->configure_serial = serial; layer_surface_configure_destroy(surface->acked_configure);
surface->current.actual_width = configure->state.actual_width; }
surface->current.actual_height = configure->state.actual_height; surface->acked_configure = configure;
wl_list_remove(&configure->link);
layer_surface_configure_destroy(configure); wl_list_init(&configure->link);
} }
static void layer_surface_handle_set_size(struct wl_client *client, static void layer_surface_handle_set_size(struct wl_client *client,
@ -170,13 +170,15 @@ static void layer_surface_resource_destroy(struct wl_resource *resource) {
} }
static bool wlr_layer_surface_state_changed(struct wlr_layer_surface *surface) { static bool wlr_layer_surface_state_changed(struct wlr_layer_surface *surface) {
if (!surface->configured) {
return true;
}
struct wlr_layer_surface_state *state; struct wlr_layer_surface_state *state;
if (wl_list_empty(&surface->configure_list)) { if (wl_list_empty(&surface->configure_list)) {
state = &surface->current; if (surface->acked_configure) {
state = &surface->acked_configure->state;
} else if (!surface->configured) {
return true;
} else {
state = &surface->current;
}
} else { } else {
struct wlr_layer_surface_configure *configure = struct wlr_layer_surface_configure *configure =
wl_container_of(surface->configure_list.prev, configure, link); wl_container_of(surface->configure_list.prev, configure, link);
@ -225,6 +227,22 @@ static void handle_wlr_surface_committed(struct wlr_surface *wlr_surface,
void *role_data) { void *role_data) {
struct wlr_layer_surface *surface = role_data; struct wlr_layer_surface *surface = role_data;
if (surface->closed) {
// Ignore commits after the compositor has closed it
return;
}
if (surface->acked_configure) {
struct wlr_layer_surface_configure *configure =
surface->acked_configure;
surface->configured = true;
surface->configure_serial = configure->serial;
surface->current.actual_width = configure->state.actual_width;
surface->current.actual_height = configure->state.actual_height;
layer_surface_configure_destroy(configure);
surface->acked_configure = NULL;
}
if (wlr_surface_has_buffer(surface->surface) && !surface->configured) { if (wlr_surface_has_buffer(surface->surface) && !surface->configured) {
wl_resource_post_error(surface->resource, wl_resource_post_error(surface->resource,
ZWLR_LAYER_SHELL_V1_ERROR_ALREADY_CONSTRUCTED, ZWLR_LAYER_SHELL_V1_ERROR_ALREADY_CONSTRUCTED,
@ -232,11 +250,6 @@ static void handle_wlr_surface_committed(struct wlr_surface *wlr_surface,
return; return;
} }
if (surface->closed) {
// Ignore commits after the compositor has closed it
return;
}
surface->current.anchor = surface->client_pending.anchor; surface->current.anchor = surface->client_pending.anchor;
surface->current.exclusive_zone = surface->client_pending.exclusive_zone; surface->current.exclusive_zone = surface->client_pending.exclusive_zone;
surface->current.margin = surface->client_pending.margin; surface->current.margin = surface->client_pending.margin;