From: Marius Vlad <marius.vlad@collabora.com>
Date: Mon, 21 Feb 2022 12:27:28 +0000 (+0200)
Subject: layout: Address all other callsites w/ dirty+surface damage
X-Git-Tag: lamprey_12.1.14
X-Git-Url: https://gerrit.automotivelinux.org/gerrit/gitweb?a=commitdiff_plain;ds=sidebyside;h=6c27f9a1073a9fb5ad2f574f3c588040829eb879;hp=b7567f17683b60cd5b8c48d0756b261737193a3d;p=src%2Fagl-compositor.git

layout: Address all other callsites w/ dirty+surface damage

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)
---

diff --git a/src/layout.c b/src/layout.c
index aa32415..0356d01 100644
--- a/src/layout.c
+++ b/src/layout.c
@@ -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);
 	}
 }