e_policy_visibility: remove norender push/pop 35/249735/3
authorChangyeon Lee <cyeon.lee@samsung.com>
Wed, 16 Dec 2020 08:50:49 +0000 (17:50 +0900)
committerchangyeon lee <cyeon.lee@samsung.com>
Mon, 21 Dec 2020 02:57:31 +0000 (02:57 +0000)
norender is pushed for keeping the frame of client.
but it isn't needed because the evas object isn't changed
untill job is executed.

Change-Id: Ifff542739fe8cbf5fe5fdb94241e148ed0259896

src/bin/e_policy_visibility.c

index fd4fe5e51d36e974a61fd1dd54deeb36a5c498f9..ab3a8857a716259d5aae922b43f27158a03583a1 100644 (file)
@@ -629,12 +629,6 @@ _e_vis_job_add(E_Vis_Client *vc, E_Vis_Job_Type type, Ecore_Task_Cb timeout_func
    job->timer = ecore_timer_add(E_VIS_TIMEOUT, timeout_func, job);
    VS_INF(vc->ec, "NEW JOB:%p, type:%d", job, type);
 
-   if ((job->type == E_VIS_JOB_TYPE_LOWER) ||
-       (job->type == E_VIS_JOB_TYPE_HIDE) ||
-       (job->type == E_VIS_JOB_TYPE_ICONIFY) ||
-       (job->type == E_VIS_JOB_TYPE_LAYER_LOWER))
-     e_comp_canvas_norender_push();
-
    return EINA_TRUE;
 }
 
@@ -645,11 +639,6 @@ _e_vis_job_del(Eina_Clist *elem)
 
    _e_vis_clist_unlink(elem);
    job = EINA_CLIST_ENTRY(elem, E_Vis_Job, entry);
-   if ((job->type == E_VIS_JOB_TYPE_LOWER) ||
-       (job->type == E_VIS_JOB_TYPE_HIDE) ||
-       (job->type == E_VIS_JOB_TYPE_ICONIFY) ||
-       (job->type == E_VIS_JOB_TYPE_LAYER_LOWER))
-     e_comp_canvas_norender_pop();
 
    VS_INF(job->vc->ec, "FREE JOB:%p, type:%d", job, job->type);
    E_FREE_FUNC(job->timer, ecore_timer_del);
@@ -675,14 +664,7 @@ _e_vis_job_exec(Eina_Clist *elem)
    _e_vis_clist_unlink(elem);
    job = EINA_CLIST_ENTRY(elem, E_Vis_Job, entry);
    if (job->deleted)
-     {
-        if ((job->type == E_VIS_JOB_TYPE_LOWER) ||
-            (job->type == E_VIS_JOB_TYPE_HIDE) ||
-            (job->type == E_VIS_JOB_TYPE_ICONIFY) ||
-            (job->type == E_VIS_JOB_TYPE_LAYER_LOWER))
-          e_comp_canvas_norender_pop();
-        goto end;
-     }
+     goto end;
 
    VS_INF(job->vc->ec, "EXEC JOB:%p, type:%d (is_del ec:%d)", job, job->type, e_object_is_del(E_OBJECT(job->vc->ec)));
    /* After calling the function below, ec may have been deleted.
@@ -1621,7 +1603,6 @@ _e_vis_ec_job_exec(E_Client *ec, E_Vis_Job_Type type)
          ec->exp_iconify.not_raise = 0;
          break;
       case E_VIS_JOB_TYPE_LOWER:
-         e_comp_canvas_norender_pop();
          if (ec)
            {
               e_client_lower(ec);
@@ -1630,15 +1611,12 @@ _e_vis_ec_job_exec(E_Client *ec, E_Vis_Job_Type type)
            }
          break;
       case E_VIS_JOB_TYPE_HIDE:
-         e_comp_canvas_norender_pop();
          if (ec) evas_object_hide(ec->frame);
          break;
       case E_VIS_JOB_TYPE_LAYER_LOWER:
-         e_comp_canvas_norender_pop();
          if (vc) e_client_layer_set(ec, vc->layer);
          break;
       case E_VIS_JOB_TYPE_ICONIFY:
-         e_comp_canvas_norender_pop();
          e_client_iconify(ec);
          break;
       case E_VIS_JOB_TYPE_SHOW: