e_policy_visibility: remove _e_vis_ec_foreground_check function

there was a bug that the black screen was shown when multi windows were destroyed.
we checked activity window in _e_vis_ec_above_is_non_alpla_visible,
so we can remove _e_vis_ec_foreground_check function.

Change-Id: I03eac77f7a4d89a6989284c0a873cee6afe9aa58

diff --git a/src/bin/e_policy_visibility.c b/src/bin/e_policy_visibility.c
index 0d87a229aaf9e5e89349fb815763c399fd3855de..167322973b2f596be0a5faea8253e913c800e267 100644 (file)
--- a/src/bin/e_policy_visibility.c
+++ b/src/bin/e_policy_visibility.c
@@ -1518,26 +1518,6 @@ _e_vis_ec_job_exec(E_Client *ec, E_Vis_Job_Type type)
      }
 }
 
-static Eina_Bool
-_e_vis_ec_foreground_check(E_Client *ec, Eina_Bool with_transients)
-{
-   E_Client *child;
-   Eina_List *l;
-
-   if (pol_vis->activity == ec)
-     return EINA_TRUE;
-   else if (with_transients)
-     {
-        EINA_LIST_FOREACH(ec->transients, l, child)
-          {
-             if (pol_vis->activity != child) continue;
-             return EINA_TRUE;
-          }
-     }
-
-   return EINA_FALSE;
-}
-
 static Eina_Bool
 _e_vis_ec_above_is_non_alpha_visible(E_Client *ec, Eina_Bool check_child)
 {
@@ -1836,13 +1816,6 @@ _e_vis_intercept_hide(void *data EINA_UNUSED, E_Client *ec)
         return EINA_FALSE;
      }
 
-   /* find activity client among the clients to be lower */
-   if (!_e_vis_ec_foreground_check(ec, !!e_config->transient.raise))
-     {
-        VS_INF(ec, "NO activity clients");
-        return EINA_TRUE;
-     }
-
    if (_e_vis_ec_above_is_non_alpha_visible(ec, EINA_FALSE))
      {
         VS_DBG(ec, "Obscured by above window.");
@@ -1983,13 +1956,6 @@ e_policy_visibility_client_lower(E_Client *ec)
    /* if vc has job grab, release them */
    _e_vis_client_grab_cancel(vc);
 
-   /* find activity client among the clients to be lower */
-   if (!_e_vis_ec_foreground_check(ec, !!e_config->transient.lower))
-     {
-        VS_INF(ec, "NO activity clients");
-        return EINA_FALSE;
-     }
-
    if (_e_vis_ec_above_is_non_alpha_visible(ec, EINA_TRUE))
      {
         VS_DBG(ec, "Obscured by above window.");
@@ -2020,13 +1986,6 @@ e_policy_visibility_client_iconify(E_Client *ec)
 
    if (ec->iconic) return EINA_FALSE;
 
-   /* find activity client among the clients to be lower */
-   if (!_e_vis_ec_foreground_check(ec, !!e_config->transient.iconify))
-     {
-        VS_INF(ec, "NO activity clients");
-        return EINA_FALSE;
-     }
-
    if (_e_vis_ec_above_is_non_alpha_visible(ec, EINA_TRUE))
      {
         VS_DBG(ec, "Obscured by above window.");
@@ -2126,13 +2085,6 @@ e_policy_visibility_client_layer_lower(E_Client *ec, E_Layer layer)
    /* if vc has job grab, release them */
    _e_vis_client_grab_cancel(vc);
 
-   /* find activity client among the clients to be lower */
-   if (!_e_vis_ec_foreground_check(ec, !!e_config->transient.lower))
-     {
-        VS_INF(ec, "NO activity clients");
-        return EINA_FALSE;
-     }
-
    if (_e_vis_ec_above_is_non_alpha_visible(ec, EINA_TRUE))
      {
         VS_DBG(ec, "Obscured by above window.");