ecore_wl2: do not destroy tizen_position in ecore_wl2_window_hide 72/272372/1
authorDoyoun Kang <doyoun.kang@samsung.com>
Tue, 15 Mar 2022 09:43:32 +0000 (18:43 +0900)
committerDoyoun Kang <doyoun.kang@samsung.com>
Tue, 15 Mar 2022 09:49:03 +0000 (18:49 +0900)
There was a bug that the window's position was wrong if a window called
ecore_wl2_window_position_set after calling ecore_wl2_window_hide.
In this case, the display server didn't get an information about new position
because the tizen_position was destroyed. When the window was shown again, the
display server sent configure event using old position not new position.
By this, window moved to wrong(old) position.

This patch resolve this problem.

@tizen_only

Change-Id: I28b377ca5fd77106a69f442e4300feb891236d84

src/lib/ecore_wl2/ecore_wl2_window.c

index 8646caa..d5c4393 100644 (file)
@@ -1298,9 +1298,6 @@ ecore_wl2_window_hide(Ecore_Wl2_Window *window)
    if (window->tz_visibility) tizen_visibility_destroy(window->tz_visibility);
    window->tz_visibility = NULL;
 
-   if (window->tz_position) tizen_position_destroy(window->tz_position);
-   window->tz_position = NULL;
-
    if (window->tz_resource) tizen_resource_destroy(window->tz_resource);
    window->tz_resource = NULL;
    //