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 8646caa3ab871427905797b4270b787f95ac9e1b..d5c4393a1b7cac4a4b41e64b5f24511cec1d7792 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;
    //