e_client: refactoring code for e_client_lower 39/261439/3
authorDoyoun Kang <doyoun.kang@samsung.com>
Mon, 19 Jul 2021 04:00:56 +0000 (13:00 +0900)
committerDoyoun Kang <doyoun.kang@samsung.com>
Mon, 19 Jul 2021 04:42:12 +0000 (13:42 +0900)
we extract method (_lower_belong_to_parent) from e_client_lower.

Change-Id: I2543abfbf16de6d052ecb8f5cf76413b0e98fb3d

src/bin/e_client.c

index aa0193e63fb55ab689ffc0e2d6c202fbfe74fdb5..232261550dff0518a965b10e3324ade6bfe3136b 100644 (file)
@@ -8218,6 +8218,19 @@ e_client_raise(E_Client *ec)
      evas_object_raise(ec->frame);
 }
 
+static void
+_lower_between_sibling_on_parent(E_Client *ec)
+{
+   ELOGF("POL", "LOWER child window between sibling... So, stack above on the parent (win:%zx, ec:%p)", ec, e_client_util_win_get(ec->parent), ec->parent);
+   e_client_stack_above(ec, ec->parent);
+}
+
+static void
+_lower_belong_to_parent(E_Client *ec)
+{
+   _lower_between_sibling_on_parent(ec);
+}
+
 E_API void
 e_client_lower(E_Client *ec)
 {
@@ -8234,17 +8247,10 @@ e_client_lower(E_Client *ec)
           }
      }
 
-   if (ec->parent)
-     {
-        if (ec->transient_for_always_on_parent)
-          {
-             ELOGF("POL", "LOWER child window... So, stack above on the parent (win:%zx, ec:%p)", ec, e_client_util_win_get(ec->parent), ec->parent);
-             e_client_stack_above(ec, ec->parent);
-             return;
-          }
-     }
-
-   evas_object_lower(ec->frame);
+   if (ec->parent && ec->transient_for_always_on_parent)
+     _lower_belong_to_parent(ec);
+   else
+     evas_object_lower(ec->frame);
 }
 
 E_API void