shell: Do not set none role for applications w/o appid 43/27443/1
authorMarius Vlad <marius.vlad@collabora.com>
Wed, 4 May 2022 08:12:54 +0000 (11:12 +0300)
committerMarius Vlad <marius.vlad@collabora.com>
Wed, 4 May 2022 08:25:15 +0000 (11:25 +0300)
With commit 'layout: Send dimensions when setting up property as
fullscreen' we made so that fullscreen applications will receive the
configure event right from the beginning but that changed a bit the way
we handled applications that do not set-up an appid before doing the
initial wl_surface.commit.

Avoid doing that such that applications do not need to go through a
resize.

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

src/shell.c

index d91570f..dc857fc 100644 (file)
@@ -638,11 +638,6 @@ ivi_check_pending_surface_desktop(struct ivi_surface *surface,
        const char *app_id =
                weston_desktop_surface_get_app_id(surface->dsurface);
 
-       if (!app_id) {
-               *role = IVI_SURFACE_ROLE_NONE;
-               return;
-       }
-
        role_pending_list = &ivi->popup_pending_apps;
        wl_list_for_each(p_popup, role_pending_list, link) {
                if (!strcmp(app_id, p_popup->app_id)) {