shell: Don't reset the activation area always 11/29011/1
authorMarius Vlad <marius.vlad@collabora.com>
Tue, 13 Jun 2023 10:47:19 +0000 (13:47 +0300)
committerMarius Vlad <marius.vlad@collabora.com>
Tue, 13 Jun 2023 10:58:28 +0000 (13:58 +0300)
Rather than doing that always, be more selective, and perform it if we
had already an application active.

Bug-AGL: SPEC-4829
Signed-off-by: Marius Vlad <marius.vlad@collabora.com>
Change-Id: I707f92d1cc1f820af493bae9daa5a948aa452ad8

src/shell.c

index 2f8a722..1aabb5a 100644 (file)
@@ -1795,18 +1795,20 @@ unbind_agl_shell(struct wl_resource *resource)
        }
 
        wl_list_for_each(output, &ivi->outputs, link) {
-               struct weston_geometry area = {};
 
                /* reset the active surf if there's one present */
                if (output->active) {
+                       struct weston_geometry area = {};
+
                        output->active->view->is_mapped = false;
                        output->active->view->surface->is_mapped = false;
 
                        weston_layer_entry_remove(&output->active->view->layer_link);
                        output->active = NULL;
+
+                       output->area_activation = area;
                }
 
-               output->area_activation = area;
                insert_black_curtain(output);
        }