gl-renderer: Account for offset in output region translation

In 55bcb93fef ("gl-renderer: Use helper for conversion to EGL rects"),
we extracted and lovingly commented the transformation from global to
output co-ordinate space used for EGL_KHR_swap_buffer_with_damage, into
a new helper function.

The commenting correctly noted the steps we need to perform the
transformation: shifting by the output's offset into global space,
followed by applying the output's scale and rotation transformations.

Unfortunately, the code did not live up to the high standards of the
comment, and forgot to translate by the output's offset. This meant that
for multiple outputs, we would probably end up with wildly out-of-bounds
co-ordinates.

Fix the code to first translate by the output's offset in global space,
ensuring that both our swap_buffers_with_damage, and our partial_update
co-ordinate sets, can spark joy for those blessed with more than one
output.

Signed-off-by: Daniel Stone <daniels@collabora.com>
dev
Daniel Stone 5 years ago committed by Daniel Stone
parent 3ea03bb85e
commit 3ebd870487
  1. 4
      libweston/renderer-gl/gl-renderer.c

@ -1401,10 +1401,12 @@ pixman_region_to_egl_y_invert(struct weston_output *output,
/* Translate from global to output co-ordinate space. */ /* Translate from global to output co-ordinate space. */
pixman_region32_init(&transformed); pixman_region32_init(&transformed);
pixman_region32_copy(&transformed, global_region);
pixman_region32_translate(&transformed, -output->x, -output->y);
weston_transformed_region(output->width, output->height, weston_transformed_region(output->width, output->height,
output->transform, output->transform,
output->current_scale, output->current_scale,
global_region, &transformed); &transformed, &transformed);
/* If we have borders drawn around the output, shift our output damage /* If we have borders drawn around the output, shift our output damage
* to account for borders being drawn around the outside, adding any * to account for borders being drawn around the outside, adding any

Loading…
Cancel
Save