Fix keyboard removal logic in _window_remove

Change-Id: I8a6322e62b3c9256b548a11bf925218582daacd1
(cherry picked from commit eddb05c76808da23d24cf5aebfdfabff84ff58de)

diff --git a/src/window_tracker.c b/src/window_tracker.c
index 37693a0d9f23cfc94c574a5bfa1d0178b8f2baf2..37bdd5d5e91fa254240789b43fc69b1f21852637 100644 (file)
--- a/src/window_tracker.c
+++ b/src/window_tracker.c
@@ -276,6 +276,7 @@ static void _window_remove(WindowTrackerData *wtd, AtspiAccessible *window)
        Eina_List *l, *l_prev;
        WindowInfo *wi = NULL;
        int removed_index = 0;
+       int keyboard_index = -1;
        EINA_LIST_REVERSE_FOREACH_SAFE(wtd->window_infos, l, l_prev, wi) {
                if (!wi) continue;
                if (atspi_accessible_is_equal(wi->window, window)) {
@@ -286,7 +287,9 @@ static void _window_remove(WindowTrackerData *wtd, AtspiAccessible *window)
                        wtd->window_infos = eina_list_remove_list(wtd->window_infos, l);
                        window_removed = EINA_TRUE;
                        break;
-               }
+               } else if (wi->keyboard_window_is)
+                       keyboard_index = removed_index;
+
                removed_index++;
        }
 
@@ -328,24 +331,10 @@ static void _window_remove(WindowTrackerData *wtd, AtspiAccessible *window)
                        DEBUG("top window: %s", id);
                        g_free(id);
                }
-       } else {
+       } else if (removed_index > 0 && keyboard_index == removed_index - 1) {
                /* The removed window could be under the keyboard window,
                in this case the keyboard window should be removed from the list */
-               if (removed_index != 0) {
-                       int count = eina_list_count(wtd->window_infos);
-                       wi = eina_list_nth(wtd->window_infos, count - removed_index - 1);
-                       if (wi) {
-                               gchar *id = atspi_accessible_get_unique_id(wi->window, NULL);
-                               DEBUG("Check upper window: %s, keyboard window: %d", id, wi->keyboard_window_is);
-                               g_free(id);
-                               if (wi->keyboard_window_is) {
-                                       wtd->window_infos = eina_list_remove(wtd->window_infos, wi);
-                                       g_object_unref(wi->window);
-                                       g_free(wi);
-                               }
-                       } else
-                               DEBUG("Current window info list is emtpy");
-               }
+               window_tracker_keyboard_window_remove(wtd);
        }
 
        gchar *id = atspi_accessible_get_unique_id(window, NULL);