From 1cea73d2d816f50222c4e27a20b3a5c08ae104df Mon Sep 17 00:00:00 2001 From: emersion Date: Sun, 4 Feb 2018 11:59:04 +0100 Subject: rootston: fix rotated views damage tracking --- rootston/output.c | 87 +++++++++++++++++++++++++++---------------------------- types/wlr_box.c | 8 ++--- 2 files changed, 46 insertions(+), 49 deletions(-) diff --git a/rootston/output.c b/rootston/output.c index 8e9e2b73..0e0c6114 100644 --- a/rootston/output.c +++ b/rootston/output.c @@ -553,31 +553,16 @@ static void damage_whole_surface(struct wlr_surface *surface, wlr_output_transformed_resolution(output->wlr_output, &ow, &oh); struct wlr_box box; - surface_intersect_output(surface, output->desktop->layout, + bool intersects = surface_intersect_output(surface, output->desktop->layout, output->wlr_output, lx, ly, rotation, &box); + if (!intersects) { + return; + } - // Take the surface damage and damage the whole surface - // The surface damage is still useful in case the surface got resized - pixman_region32_t damage; - pixman_region32_init(&damage); - pixman_region32_copy(&damage, &surface->current->surface_damage); - wlr_region_scale(&damage, &damage, output->wlr_output->scale); - pixman_region32_union_rect(&damage, &damage, 0, 0, box.width, box.height); - pixman_region32_translate(&damage, box.x, box.y); - pixman_region32_intersect_rect(&damage, &damage, 0, 0, ow, oh); - pixman_box32_t *extents = pixman_region32_extents(&damage); - struct wlr_box extents_box = { - .x = extents->x1, - .y = extents->y1, - .width = extents->x2 - extents->x1, - .height = extents->y2 - extents->y1, - }; - pixman_region32_fini(&damage); - - wlr_box_rotated_bounds(&extents_box, -rotation, &extents_box); + wlr_box_rotated_bounds(&box, -rotation, &box); pixman_region32_union_rect(&output->damage, &output->damage, - extents_box.x, extents_box.y, extents_box.width, extents_box.height); + box.x, box.y, box.width, box.height); wlr_output_schedule_frame(output->wlr_output); } @@ -616,34 +601,51 @@ void output_damage_whole_drag_icon(struct roots_output *output, static void damage_from_surface(struct wlr_surface *surface, double lx, double ly, float rotation, void *data) { struct roots_output *output = data; + struct wlr_output *wlr_output = output->wlr_output; if (!wlr_surface_has_buffer(surface)) { return; } int ow, oh; - wlr_output_transformed_resolution(output->wlr_output, &ow, &oh); + wlr_output_transformed_resolution(wlr_output, &ow, &oh); struct wlr_box box; surface_intersect_output(surface, output->desktop->layout, - output->wlr_output, lx, ly, rotation, &box); - - pixman_region32_t damage; - pixman_region32_init(&damage); - pixman_region32_copy(&damage, &surface->current->surface_damage); - wlr_region_scale(&damage, &damage, output->wlr_output->scale); - if (ceil(output->wlr_output->scale) > surface->current->scale) { - // When scaling up a surface, it'll become blurry so we need to expand - // the damage region - wlr_region_expand(&damage, &damage, - ceil(output->wlr_output->scale) - surface->current->scale); - } - pixman_region32_translate(&damage, box.x, box.y); - pixman_region32_intersect_rect(&damage, &damage, 0, 0, ow, oh); - pixman_region32_union(&output->damage, &output->damage, &damage); - pixman_region32_fini(&damage); - - wlr_output_schedule_frame(output->wlr_output); + wlr_output, lx, ly, rotation, &box); + + if (rotation == 0) { + pixman_region32_t damage; + pixman_region32_init(&damage); + pixman_region32_copy(&damage, &surface->current->surface_damage); + wlr_region_scale(&damage, &damage, wlr_output->scale); + if (ceil(wlr_output->scale) > surface->current->scale) { + // When scaling up a surface, it'll become blurry so we need to + // expand the damage region + wlr_region_expand(&damage, &damage, + ceil(wlr_output->scale) - surface->current->scale); + } + pixman_region32_translate(&damage, box.x, box.y); + pixman_region32_union(&output->damage, &output->damage, &damage); + pixman_region32_fini(&damage); + } else { + pixman_box32_t *extents = + pixman_region32_extents(&surface->current->surface_damage); + struct wlr_box damage_box = { + .x = box.x + extents->x1 * wlr_output->scale, + .y = box.y + extents->y1 * wlr_output->scale, + .width = (extents->x2 - extents->x1) * wlr_output->scale, + .height = (extents->y2 - extents->y1) * wlr_output->scale, + }; + wlr_box_rotated_bounds(&damage_box, -rotation, &damage_box); + pixman_region32_union_rect(&output->damage, &output->damage, + damage_box.x, damage_box.y, damage_box.width, damage_box.height); + } + + pixman_region32_intersect_rect(&output->damage, &output->damage, 0, 0, + ow, oh); + + wlr_output_schedule_frame(wlr_output); } void output_damage_from_view(struct roots_output *output, @@ -652,11 +654,6 @@ void output_damage_from_view(struct roots_output *output, return; } - if (view->rotation != 0) { - output_damage_whole_view(output, view); - return; - } - view_for_each_surface(view, damage_from_surface, output); } diff --git a/types/wlr_box.c b/types/wlr_box.c index a7388209..33031858 100644 --- a/types/wlr_box.c +++ b/types/wlr_box.c @@ -138,8 +138,8 @@ void wlr_box_rotated_bounds(const struct wlr_box *box, float rotation, (box->x + box->width - ox) * s + (box->y + box->height - oy) * c; - dest->x = fmin(x1, x2); - dest->width = fmax(x1, x2) - fmin(x1, x2); - dest->y = fmin(y1, y2); - dest->height = fmax(y1, y2) - fmin(y1, y2); + dest->x = floor(fmin(x1, x2)); + dest->width = ceil(fmax(x1, x2) - fmin(x1, x2)); + dest->y = floor(fmin(y1, y2)); + dest->height = ceil(fmax(y1, y2) - fmin(y1, y2)); } -- cgit v1.2.3