X-Git-Url: https://gerrit.automotivelinux.org/gerrit/gitweb?a=blobdiff_plain;f=src%2Fdesktop.c;h=a3552e2141f3e6b34598e4de268570047ab464d7;hb=07c95a07d69ea03f3bfbf21a1f737fc175b9a756;hp=7502d6cb38b5b0b913ad5dd261e789eb1eb99c55;hpb=a9ae8d75026b1fa1ef32d143da23bc8ee9aedeaa;p=src%2Fagl-compositor.git

diff --git a/src/desktop.c b/src/desktop.c
index 7502d6c..a3552e2 100644
--- a/src/desktop.c
+++ b/src/desktop.c
@@ -61,6 +61,7 @@ desktop_surface_added(struct weston_desktop_surface *dsurface, void *userdata)
 	struct weston_desktop_client *dclient;
 	struct wl_client *client;
 	struct ivi_surface *surface;
+	const char *app_id = NULL;
 
 	dclient = weston_desktop_surface_get_client(dsurface);
 	client = weston_desktop_client_get_client(dclient);
@@ -92,18 +93,37 @@ desktop_surface_added(struct weston_desktop_surface *dsurface, void *userdata)
 
 	weston_desktop_surface_set_user_data(dsurface, surface);
 
+	app_id = weston_desktop_surface_get_app_id(dsurface);
+
 	if (ivi->shell_client.ready) {
 		ivi_check_pending_desktop_surface(surface);
+		weston_log("Added surface %p, app_id %s, role %s\n", surface,
+				app_id, ivi_layout_get_surface_role_name(surface));
 	} else {
 		/*
 		 * We delay creating "normal" desktop surfaces until later, to
 		 * give the shell-client an oppurtunity to set the surface as a
 		 * background/panel.
 		 */
+		weston_log("Added surface %p, app_id %s to pending list\n",
+				surface, app_id);
 		wl_list_insert(&ivi->pending_surfaces, &surface->link);
 	}
 }
 
+static bool
+desktop_surface_check_last_remote_surfaces(struct ivi_compositor *ivi)
+{
+	int count = 0;
+	struct ivi_surface *surf;
+
+	wl_list_for_each(surf, &ivi->surfaces, link)
+		if (surf->role == IVI_SURFACE_ROLE_REMOTE)
+			count++;
+
+	return (count == 1);
+}
+
 static void
 desktop_surface_removed(struct weston_desktop_surface *dsurface, void *userdata)
 {
@@ -112,19 +132,8 @@ desktop_surface_removed(struct weston_desktop_surface *dsurface, void *userdata)
 	struct weston_surface *wsurface =
 		weston_desktop_surface_get_surface(dsurface);
 
-	struct ivi_output *output;
-
-	if (surface->role == IVI_SURFACE_ROLE_DESKTOP)
-		output = surface->desktop.last_output;
-	else if (surface->role == IVI_SURFACE_ROLE_POPUP)
-		output = surface->popup.output;
-	else if (surface->role == IVI_SURFACE_ROLE_SPLIT_H ||
-		 surface->role == IVI_SURFACE_ROLE_SPLIT_V)
-		output = surface->split.output;
-	else if (surface->role == IVI_SURFACE_ROLE_FULLSCREEN)
-		output = surface->fullscreen.output;
-	else
-		return;
+	struct ivi_output *output = ivi_layout_get_output_from_surface(surface);
+	assert(output != NULL);
 
 	/* resize the active surface to the original size */
 	if (surface->role == IVI_SURFACE_ROLE_SPLIT_H ||
@@ -144,6 +153,14 @@ desktop_surface_removed(struct weston_desktop_surface *dsurface, void *userdata)
 		weston_layer_entry_remove(&output->active->view->layer_link);
 		output->active = NULL;
 	}
+
+	/* check if there's a last 'remote' surface and insert a black
+	 * surface view if there's no background set for that output
+	 */
+	if (desktop_surface_check_last_remote_surfaces(output->ivi))
+		if (!output->background)
+			insert_black_surface(output);
+
 	if (weston_surface_is_mapped(wsurface)) {
 		weston_desktop_surface_unlink_view(surface->view);
 		weston_view_destroy(surface->view);
@@ -172,6 +189,9 @@ desktop_surface_removed(struct weston_desktop_surface *dsurface, void *userdata)
 		output->background = NULL;
 	}
 
+	weston_log("Removed surface %p, app_id %s, role %s\n", surface,
+			weston_desktop_surface_get_app_id(dsurface),
+			ivi_layout_get_surface_role_name(surface));
 	wl_list_remove(&surface->link);
 	free(surface);
 }
@@ -192,6 +212,7 @@ desktop_committed(struct weston_desktop_surface *dsurface,
 
 	switch (surface->role) {
 	case IVI_SURFACE_ROLE_DESKTOP:
+	case IVI_SURFACE_ROLE_REMOTE:
 		ivi_layout_desktop_committed(surface);
 		break;
 	case IVI_SURFACE_ROLE_PANEL: