app_tracker: set timer variable to NULL before return FALSE 21/269821/1 accepted/tizen/6.0/unified/20220126.131355 submit/tizen_6.0/20220121.012711
authorShinwoo Kim <cinoo.kim@samsung.com>
Tue, 18 Jan 2022 05:42:44 +0000 (14:42 +0900)
committerShinwoo Kim <cinoo.kim@samsung.com>
Thu, 20 Jan 2022 08:09:31 +0000 (17:09 +0900)
First of all, it is the correct way to use of timer.

If a toolkit does not support the default label,
then it is not able to make an object have focus (highlight)
because app_tracker_context_valid_check in _focus_widget returns
APP_TRACKER_CONTEXT_NOT_VALID.

Change-Id: I6b2db7b8afd7598d63820121725ac11a22281be0

src/app_tracker.c

index 1912653857eaa029c936eb27871fc354c577c8a9..2f3e7066237d74a00159bb6f884ebe8e654dc4bb 100644 (file)
@@ -72,7 +72,7 @@ static gboolean _on_timeout_rebuild_navigation_context(gpointer user_data)
 
        if (!atd->root || object_has_defunct_state(atd->root)) {
                ERROR("Cannot rebuild navigation context. Active window is defunct.");
-               return FALSE;
+               goto end;
        }
 
        if (atd->view_content_changed_ecd && atd->view_content_changed_ecd->func) {
@@ -83,7 +83,7 @@ static gboolean _on_timeout_rebuild_navigation_context(gpointer user_data)
 
                if (!dli) {
                        ERROR("Default label info is NULL!");
-                       return FALSE;
+                       goto end;
                }
 
                WindowActivateInfoType window_activate_info_type =
@@ -131,6 +131,8 @@ static gboolean _on_timeout_rebuild_navigation_context(gpointer user_data)
                        g_object_unref(dli->obj);
                g_free(dli);
        }
+
+end:
        atd->timer = 0;
        DEBUG("END");
        return FALSE;