From: Doyoun Kang <doyoun.kang@samsung.com>
Date: Tue, 15 Mar 2022 09:43:32 +0000 (+0900)
Subject: ecore_wl2: do not destroy tizen_position in ecore_wl2_window_hide
X-Git-Tag: accepted/tizen/unified/20220325.133343~7
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=c22287749a18e4a21f77368d526c7bf75923edbd;p=platform%2Fupstream%2Fefl.git

ecore_wl2: do not destroy tizen_position in ecore_wl2_window_hide

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
---

diff --git a/src/lib/ecore_wl2/ecore_wl2_window.c b/src/lib/ecore_wl2/ecore_wl2_window.c
index 8646caa..d5c4393 100644
--- a/src/lib/ecore_wl2/ecore_wl2_window.c
+++ b/src/lib/ecore_wl2/ecore_wl2_window.c
@@ -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;
    //