app_tracker: set timer variable to NULL before return FALSE 39/269539/2 accepted/tizen/unified/20220127.133339 submit/tizen/20220121.012748
authorShinwoo Kim <cinoo.kim@samsung.com>
Tue, 18 Jan 2022 05:42:44 +0000 (14:42 +0900)
committerShinwoo Kim <cinoo.kim@samsung.com>
Tue, 18 Jan 2022 05:51:27 +0000 (14:51 +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 272baa791e986d74306bb0527025b308bfe22a5a..ef4c055ff66a9fa2703dc98bec50ed653a717457 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;