shell: Determine the role type 25/24525/7
authorMarius Vlad <marius.vlad@collabora.com>
Sat, 16 May 2020 13:18:50 +0000 (16:18 +0300)
committerMarius Vlad <marius.vlad@collabora.com>
Mon, 8 Jun 2020 14:26:36 +0000 (17:26 +0300)
Now that we have the ability to determine the role type between
pop-up diaglos, full-screen, or split ones use the wrappers installed
previously to make use of them.

Keep the same functionality by activating the desktop surface if we
determine we have one.

Bug-AGL: SPEC-3334

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

src/shell.c

index c132ae5..df8ea84 100644 (file)
@@ -276,8 +276,10 @@ ivi_check_pending_desktop_surface(struct ivi_surface *surface)
                return;
        }
 
-       /* if we end up here means we have a regular desktop app */
+       /* if we end up here means we have a regular desktop app and
+        * try to activate it */
        ivi_set_desktop_surface(surface);
+       ivi_layout_desktop_committed(surface);
 }
 
 void
@@ -525,13 +527,7 @@ shell_ready(struct wl_client *client, struct wl_resource *shell_res)
 
        wl_list_for_each_safe(surface, tmp, &ivi->pending_surfaces, link) {
                wl_list_remove(&surface->link);
-
-               if (ivi_check_pending_desktop_surface_popup(surface)) {
-                       ivi_set_desktop_surface_popup(surface);
-               } else {
-                       ivi_set_desktop_surface(surface);
-                       ivi_layout_desktop_committed(surface);
-               }
+               ivi_check_pending_desktop_surface(surface);
        }
 }