From: Marius Vlad <marius.vlad@collabora.com>
Date: Sun, 12 Jul 2020 16:26:22 +0000 (+0300)
Subject: shell: Assume consistency with the desktop roles
X-Git-Tag: 9.99.3~2
X-Git-Url: https://gerrit.automotivelinux.org/gerrit/gitweb?a=commitdiff_plain;h=35f6733fb39bc83cc16652a3da849d9f383263ef;p=src%2Fagl-compositor.git

shell: Assume consistency with the desktop roles

While looking into activation by default of other surfaces, noticed that
we're missing consistency on how we handle other types of surface roles.

This patch, activate all other surfaces by default when starting up,
even if they are started before or after the client shell.
Expecting and assuming the same behaviour is important into trying
to establish some common work-flow for applications.

Bug-AGL: SPEC-3494

Signed-off-by: Marius Vlad <marius.vlad@collabora.com>
Change-Id: I90e764c6c3a7ed543934ee6f68fa716d90ff151e
---

diff --git a/src/shell.c b/src/shell.c
index 4886a0a..2978cc3 100644
--- a/src/shell.c
+++ b/src/shell.c
@@ -366,24 +366,28 @@ ivi_check_pending_desktop_surface(struct ivi_surface *surface)
 	ret = ivi_check_pending_desktop_surface_popup(surface);
 	if (ret) {
 		ivi_set_desktop_surface_popup(surface);
+		ivi_layout_popup_committed(surface);
 		return;
 	}
 
 	ret = ivi_check_pending_desktop_surface_split(surface);
 	if (ret) {
 		ivi_set_desktop_surface_split(surface);
+		ivi_layout_split_committed(surface);
 		return;
 	}
 
 	ret = ivi_check_pending_desktop_surface_fullscreen(surface);
 	if (ret) {
 		ivi_set_desktop_surface_fullscreen(surface);
+		ivi_layout_fullscreen_committed(surface);
 		return;
 	}
 
 	ret = ivi_check_pending_desktop_surface_remote(surface);
 	if (ret) {
 		ivi_set_desktop_surface_remote(surface);
+		ivi_layout_desktop_committed(surface);
 		return;
 	}