e_policy_visibility: refactoring code

- Rename internal value in _e_vis_ec_below_uniconify
- Add some logs

Change-Id: I7b470c6ada7129436578613193e9302b604fa6e4

diff --git a/src/bin/e_policy_visibility.c b/src/bin/e_policy_visibility.c
index 358d0af..5d9055b 100644 (file)
--- a/src/bin/e_policy_visibility.c
+++ b/src/bin/e_policy_visibility.c
@@ -956,7 +956,7 @@ _e_vis_client_job_exec(E_Vis_Client *vc, E_Vis_Job_Type type)
 static void
 _e_vis_client_grab_add(E_Vis_Client *vc, E_Vis_Grab *grab)
 {
-   VS_INF(vc->ec, "Add Client Visibility Grab: %s", grab->name);
+   VS_INF(vc->ec, "Add Client Visibility Grab: %p(%s)", grab, grab->name);
 
    vc->job.grab_list = eina_list_append(vc->job.grab_list, grab);
 }
@@ -964,7 +964,7 @@ _e_vis_client_grab_add(E_Vis_Client *vc, E_Vis_Grab *grab)
 static void
 _e_vis_client_grab_remove(E_Vis_Client *vc, E_Vis_Grab *grab)
 {
-   VS_INF(vc->ec, "Remove Client Visibility Grab: %s", grab->name);
+   VS_INF(vc->ec, "Remove Client Visibility Grab: %p(%s)", grab, grab->name);
 
    if (!vc->job.grab_list)
      {
@@ -1168,7 +1168,10 @@ _e_vis_client_del(E_Vis_Client *vc)
 
    E_FREE_FUNC(vc->grab, _e_vis_grab_release);
    E_LIST_REVERSE_FREE(vc->job.grab_list, grab)
-      grab->deleted = 1;
+     {
+        VS_INF(vc->ec, "CRI... Not handled grab (%p)", grab);
+        grab->deleted = 1;
+     }
 
    /* if it's intended normal operation, there is no job to delete. */
    _e_vis_job_del_by_client(vc);
@@ -1853,7 +1856,7 @@ _e_vis_ec_below_uniconify(E_Client *ec, E_Pol_Vis_Type above_vis_type)
 {
    Eina_List *below_list = NULL;
    Eina_Bool ret = EINA_FALSE;
-   E_Vis_Client *below;
+   E_Vis_Client *below_vc;
    Eina_Bool send_vis;
 
    if (!ec) return EINA_FALSE;
@@ -1880,19 +1883,19 @@ _e_vis_ec_below_uniconify(E_Client *ec, E_Pol_Vis_Type above_vis_type)
         else
           send_vis = EINA_TRUE;
 
-        EINA_LIST_FREE(below_list, below)
+        EINA_LIST_FREE(below_list, below_vc)
           {
              Eina_Bool job_added = EINA_FALSE;
-             E_Client *below_ec = below->ec;
+             E_Client *below_ec = below_vc->ec;
 
              if (below_ec == ec->parent)
                {
                   /* Check if its parent is waiting for a child's uniconify.
                    * if so cancel the waiting now.
                    */
-                  if (below->state == E_VIS_ICONIFY_STATE_RUNNING_UNICONIFY_WAITING_FOR_CHILD)
+                  if (below_vc->state == E_VIS_ICONIFY_STATE_RUNNING_UNICONIFY_WAITING_FOR_CHILD)
                     {
-                       E_FREE_FUNC(below->grab, _e_vis_grab_release);
+                       E_FREE_FUNC(below_vc->grab, _e_vis_grab_release);
                        continue;
                     }
 
@@ -1900,7 +1903,7 @@ _e_vis_ec_below_uniconify(E_Client *ec, E_Pol_Vis_Type above_vis_type)
                   e_vis_client_send_pre_visibility_event(below_ec);
                }
 
-             job_added = _e_vis_client_add_uniconify_render_pending(below, E_VIS_JOB_TYPE_UNICONIFY_BY_VISIBILITY, 0, send_vis);
+             job_added = _e_vis_client_add_uniconify_render_pending(below_vc, E_VIS_JOB_TYPE_UNICONIFY_BY_VISIBILITY, 0, send_vis);
 
              if (!job_added)
                {