blankie
/
wlroots
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

compositor: damage the whole buffer on viewport src change 

wp_viewporter protocol doesn't seem to say anything about damage, but
Firefox assumes that wp_viewport::set_source alone is enough to damage
the whole surface, and that assumption kinda makes sense, so let's do
that.

(cherry picked from commit da2491d4163e1d8e627d00c8ae594c7f8003472e)
This commit is contained in:
Kirill Primak 2022-01-22 11:35:22 +03:00 committed by Simon Ser
parent b0fee56974
commit 304c61307a
1 changed files with 7 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
types/wlr_surface.c
View File

@ -207,8 +207,12 @@ static void surface_update_damage(pixman_region32_t *buffer_damage,
pixman_region32_clear(buffer_damage); pixman_region32_clear(buffer_damage);
if (pending->width != current->width || if (pending->width != current->width ||
pending->height != current->height) { pending->height != current->height ||
// Damage the whole buffer on resize pending->viewport.src.x != current->viewport.src.x ||
pending->viewport.src.y != current->viewport.src.y ||
pending->viewport.src.width != current->viewport.src.width ||
pending->viewport.src.height != current->viewport.src.height) {
// Damage the whole buffer on resize or viewport source box change
pixman_region32_union_rect(buffer_damage, buffer_damage, 0, 0, pixman_region32_union_rect(buffer_damage, buffer_damage, 0, 0,
pending->buffer_width, pending->buffer_height); pending->buffer_width, pending->buffer_height);
} else { } else {
@ -491,7 +495,7 @@ static void collect_subsurface_damage_iter(struct wlr_surface *surface,
static void subsurface_parent_commit(struct wlr_subsurface *subsurface) { static void subsurface_parent_commit(struct wlr_subsurface *subsurface) {
struct wlr_surface *surface = subsurface->surface; struct wlr_surface *surface = subsurface->surface;
bool moved = subsurface->current.x != subsurface->pending.x || bool moved = subsurface->current.x != subsurface->pending.x ||
subsurface->current.y != subsurface->pending.y; subsurface->current.y != subsurface->pending.y;
if (subsurface->mapped && moved) { if (subsurface->mapped && moved) {