layout: Address all other callsites w/ dirty+surface damage 30/27530/1 lamprey 12.1.10 12.1.11 12.1.12 12.1.13 12.1.14 12.1.15 12.1.16 12.1.17 12.1.18 12.1.19 12.1.20 12.1.4 12.1.5 12.1.6 12.1.7 12.1.8 12.1.9 lamprey/12.1.10 lamprey/12.1.11 lamprey/12.1.12 lamprey/12.1.13 lamprey/12.1.14 lamprey/12.1.15 lamprey/12.1.16 lamprey/12.1.17 lamprey/12.1.18 lamprey/12.1.19 lamprey/12.1.20 lamprey/12.1.4 lamprey/12.1.5 lamprey/12.1.6 lamprey/12.1.7 lamprey/12.1.8 lamprey/12.1.9 lamprey_12.1.10 lamprey_12.1.11 lamprey_12.1.12 lamprey_12.1.13 lamprey_12.1.14 lamprey_12.1.15 lamprey_12.1.16 lamprey_12.1.17 lamprey_12.1.18 lamprey_12.1.19 lamprey_12.1.20 lamprey_12.1.4 lamprey_12.1.5 lamprey_12.1.6 lamprey_12.1.7 lamprey_12.1.8 lamprey_12.1.9
authorMarius Vlad <marius.vlad@collabora.com>
Mon, 21 Feb 2022 12:27:28 +0000 (14:27 +0200)
committerMarius Vlad <marius.vlad@collabora.com>
Thu, 26 May 2022 10:54:46 +0000 (13:54 +0300)
This is just a missing left-over 'layout: Inflict damage on all
subsurfaces'. As we have different paths for other types of surface
roles, address them as well.

Bug-AGL: SPEC-4262

Signed-off-by: Marius Vlad <marius.vlad@collabora.com>
Change-Id: I2b68bf986ad694b0e31396edc24c54cf1dc4d834
(cherry picked from commit cec4cf8fe409cae5618e81e4a3b6dd11cc8b2aa9)

src/layout.c

index aa32415..0356d01 100644 (file)
@@ -427,8 +427,8 @@ ivi_layout_fullscreen_committed(struct ivi_surface *surface)
        weston_view_set_position(view, woutput->x, woutput->y);
        weston_layer_entry_insert(&ivi->fullscreen.view_list, &view->layer_link);
 
-       weston_view_update_transform(view);
-       weston_view_damage_below(view);
+       weston_view_geometry_dirty(view);
+       weston_surface_damage(view->surface);
 
        wsurface->is_mapped = true;
        surface->view->is_mapped = true;
@@ -456,8 +456,9 @@ ivi_layout_desktop_resize(struct ivi_surface *surface,
                                        width, height);
 
        weston_view_set_position(view, x, y);
-       weston_view_update_transform(view);
-       weston_view_damage_below(view);
+
+       weston_view_geometry_dirty(view);
+       weston_surface_damage(view->surface);
 }
 
 void
@@ -541,8 +542,8 @@ ivi_layout_split_committed(struct ivi_surface *surface)
        weston_view_set_position(view, x, y);
        weston_layer_entry_insert(&ivi->normal.view_list, &view->layer_link);
 
-       weston_view_update_transform(view);
-       weston_view_damage_below(view);
+       weston_view_geometry_dirty(view);
+       weston_surface_damage(view->surface);
 
        wsurface->is_mapped = true;
        surface->view->is_mapped = true;
@@ -606,8 +607,8 @@ ivi_layout_popup_committed(struct ivi_surface *surface)
 
        weston_layer_entry_insert(&ivi->popup.view_list, &view->layer_link);
 
-       weston_view_update_transform(view);
-       weston_view_damage_below(view);
+       weston_view_geometry_dirty(view);
+       weston_surface_damage(view->surface);
 
        wsurface->is_mapped = true;
        surface->view->is_mapped = true;
@@ -843,7 +844,8 @@ ivi_layout_deactivate(struct ivi_compositor *ivi, const char *app_id)
                                view->surface->is_mapped = false;
 
                                weston_layer_entry_remove(&view->layer_link);
-                               weston_view_damage_below(view);
+                               weston_view_geometry_dirty(view);
+                               weston_surface_damage(view->surface);
                                ivi_output->active = NULL;
                        }
                } else {
@@ -859,6 +861,7 @@ ivi_layout_deactivate(struct ivi_compositor *ivi, const char *app_id)
                struct weston_view *view  = surf->view;
 
                weston_layer_entry_remove(&view->layer_link);
-               weston_view_damage_below(view);
+               weston_view_geometry_dirty(view);
+               weston_surface_damage(view->surface);
        }
 }