From 9af8993332a053a42ff87e84ad050585835eb6f9 Mon Sep 17 00:00:00 2001
From: Carsten Haitzler <raster@rasterman.com>
Date: Mon, 9 Nov 2009 15:18:37 +0000
Subject: [PATCH] god damnt that took some time! fixed some bugs with map +
 smart objcs + repainting etc. etc. - seems to be working now in complex smart
 obj tests and simple ones.

SVN revision: 43584
---
 legacy/evas/src/lib/canvas/evas_map.c         |   2 -
 .../lib/canvas/evas_object_gradient2_linear.c |   3 +-
 .../lib/canvas/evas_object_gradient2_radial.c |   3 +-
 .../evas/src/lib/canvas/evas_object_image.c   |   3 +-
 legacy/evas/src/lib/canvas/evas_object_line.c |   3 +-
 .../evas/src/lib/canvas/evas_object_polygon.c |   3 +-
 .../src/lib/canvas/evas_object_rectangle.c    |   3 +-
 .../evas/src/lib/canvas/evas_object_smart.c   |   3 +-
 legacy/evas/src/lib/canvas/evas_object_text.c |   3 +-
 .../src/lib/canvas/evas_object_textblock.c    |   3 +-
 legacy/evas/src/lib/canvas/evas_render.c      | 197 ++++++++++--------
 legacy/evas/src/lib/include/evas_inline.x     |   5 +-
 12 files changed, 132 insertions(+), 99 deletions(-)

diff --git a/legacy/evas/src/lib/canvas/evas_map.c b/legacy/evas/src/lib/canvas/evas_map.c
index 0b33e6daf1..43162ab29d 100644
--- a/legacy/evas/src/lib/canvas/evas_map.c
+++ b/legacy/evas/src/lib/canvas/evas_map.c
@@ -30,7 +30,6 @@ _evas_map_calc_geom_change(Evas_Object *obj)
      }
    evas_object_inform_call_move(obj);
    evas_object_inform_call_resize(obj);
-   obj->changed = 1;
 }
 
 static void
@@ -288,7 +287,6 @@ evas_object_map_enable_set(Evas_Object *obj, Eina_Bool enabled)
      {
         if (obj->cur.map)
           {
-             obj->prev.geometry = obj->cur.map->normal_geometry;
              _evas_map_calc_geom_change(obj);
           }
      }
diff --git a/legacy/evas/src/lib/canvas/evas_object_gradient2_linear.c b/legacy/evas/src/lib/canvas/evas_object_gradient2_linear.c
index 5150cb1d18..88fc016cfb 100644
--- a/legacy/evas/src/lib/canvas/evas_object_gradient2_linear.c
+++ b/legacy/evas/src/lib/canvas/evas_object_gradient2_linear.c
@@ -352,7 +352,8 @@ evas_object_gradient2_linear_render_pre(Evas_Object *obj)
 	evas_object_render_pre_visible_change(&obj->layer->evas->clip_changes, obj, is_v, was_v);
 	goto done;
      }
-   if (obj->cur.map != obj->prev.map)
+   if ((obj->cur.map != obj->prev.map) ||
+       (obj->cur.usemap != obj->prev.usemap))
      {
 	evas_object_render_pre_prev_cur_add(&obj->layer->evas->clip_changes, obj);
         goto done;
diff --git a/legacy/evas/src/lib/canvas/evas_object_gradient2_radial.c b/legacy/evas/src/lib/canvas/evas_object_gradient2_radial.c
index c43c46e4b0..01aadb3ebd 100644
--- a/legacy/evas/src/lib/canvas/evas_object_gradient2_radial.c
+++ b/legacy/evas/src/lib/canvas/evas_object_gradient2_radial.c
@@ -349,7 +349,8 @@ evas_object_gradient2_radial_render_pre(Evas_Object *obj)
 	evas_object_render_pre_visible_change(&obj->layer->evas->clip_changes, obj, is_v, was_v);
 	goto done;
      }
-   if (obj->cur.map != obj->prev.map)
+   if ((obj->cur.map != obj->prev.map) ||
+       (obj->cur.usemap != obj->prev.usemap))
      {
 	evas_object_render_pre_prev_cur_add(&obj->layer->evas->clip_changes, obj);
         goto done;
diff --git a/legacy/evas/src/lib/canvas/evas_object_image.c b/legacy/evas/src/lib/canvas/evas_object_image.c
index 107e256709..42e3c55cdb 100644
--- a/legacy/evas/src/lib/canvas/evas_object_image.c
+++ b/legacy/evas/src/lib/canvas/evas_object_image.c
@@ -2557,7 +2557,8 @@ evas_object_image_render_pre(Evas_Object *obj)
 	evas_object_render_pre_visible_change(&obj->layer->evas->clip_changes, obj, is_v, was_v);
 	if (!o->pixel_updates) goto done;
      }
-   if (obj->cur.map != obj->prev.map)
+   if ((obj->cur.map != obj->prev.map) ||
+       (obj->cur.usemap != obj->prev.usemap))
      {
 	evas_object_render_pre_prev_cur_add(&obj->layer->evas->clip_changes, obj);
         goto done;
diff --git a/legacy/evas/src/lib/canvas/evas_object_line.c b/legacy/evas/src/lib/canvas/evas_object_line.c
index f6343e8f33..138ad20aee 100644
--- a/legacy/evas/src/lib/canvas/evas_object_line.c
+++ b/legacy/evas/src/lib/canvas/evas_object_line.c
@@ -343,7 +343,8 @@ evas_object_line_render_pre(Evas_Object *obj)
 	evas_object_render_pre_visible_change(&obj->layer->evas->clip_changes, obj, is_v, was_v);
 	goto done;
      }
-   if (obj->cur.map != obj->prev.map)
+   if ((obj->cur.map != obj->prev.map) ||
+       (obj->cur.usemap != obj->prev.usemap))
      {
 	evas_object_render_pre_prev_cur_add(&obj->layer->evas->clip_changes, obj);
         goto done;
diff --git a/legacy/evas/src/lib/canvas/evas_object_polygon.c b/legacy/evas/src/lib/canvas/evas_object_polygon.c
index 5f48f5d8f6..698316c2dc 100644
--- a/legacy/evas/src/lib/canvas/evas_object_polygon.c
+++ b/legacy/evas/src/lib/canvas/evas_object_polygon.c
@@ -358,7 +358,8 @@ evas_object_polygon_render_pre(Evas_Object *obj)
 	evas_object_render_pre_visible_change(&obj->layer->evas->clip_changes, obj, is_v, was_v);
 	goto done;
      }
-   if (obj->cur.map != obj->prev.map)
+   if ((obj->cur.map != obj->prev.map) ||
+       (obj->cur.usemap != obj->prev.usemap))
      {
 	evas_object_render_pre_prev_cur_add(&obj->layer->evas->clip_changes, obj);
         goto done;
diff --git a/legacy/evas/src/lib/canvas/evas_object_rectangle.c b/legacy/evas/src/lib/canvas/evas_object_rectangle.c
index b7593c3e09..9e2dd42776 100644
--- a/legacy/evas/src/lib/canvas/evas_object_rectangle.c
+++ b/legacy/evas/src/lib/canvas/evas_object_rectangle.c
@@ -207,7 +207,8 @@ evas_object_rectangle_render_pre(Evas_Object *obj)
 	evas_object_render_pre_visible_change(&obj->layer->evas->clip_changes, obj, is_v, was_v);
 	goto done;
      }
-   if (obj->cur.map != obj->prev.map)
+   if ((obj->cur.map != obj->prev.map) ||
+       (obj->cur.usemap != obj->prev.usemap))
      {
 	evas_object_render_pre_prev_cur_add(&obj->layer->evas->clip_changes, obj);
         goto done;
diff --git a/legacy/evas/src/lib/canvas/evas_object_smart.c b/legacy/evas/src/lib/canvas/evas_object_smart.c
index 6b7941b967..185bb97f69 100644
--- a/legacy/evas/src/lib/canvas/evas_object_smart.c
+++ b/legacy/evas/src/lib/canvas/evas_object_smart.c
@@ -819,7 +819,8 @@ static void
 evas_object_smart_render_pre(Evas_Object *obj)
 {
    if (obj->pre_render_done) return;
-   if (obj->cur.map != obj->prev.map)
+   if ((obj->cur.map != obj->prev.map) ||
+       (obj->cur.usemap != obj->prev.usemap))
      {
 	evas_object_render_pre_prev_cur_add(&obj->layer->evas->clip_changes, obj);
         goto done;
diff --git a/legacy/evas/src/lib/canvas/evas_object_text.c b/legacy/evas/src/lib/canvas/evas_object_text.c
index eae61d75f6..9042060422 100644
--- a/legacy/evas/src/lib/canvas/evas_object_text.c
+++ b/legacy/evas/src/lib/canvas/evas_object_text.c
@@ -1637,7 +1637,8 @@ evas_object_text_render_pre(Evas_Object *obj)
 	evas_object_render_pre_visible_change(&obj->layer->evas->clip_changes, obj, is_v, was_v);
 	goto done;
      }
-   if (obj->cur.map != obj->prev.map)
+   if ((obj->cur.map != obj->prev.map) ||
+       (obj->cur.usemap != obj->prev.usemap))
      {
 	evas_object_render_pre_prev_cur_add(&obj->layer->evas->clip_changes, obj);
         goto done;
diff --git a/legacy/evas/src/lib/canvas/evas_object_textblock.c b/legacy/evas/src/lib/canvas/evas_object_textblock.c
index d410e0a22c..2feb34483d 100644
--- a/legacy/evas/src/lib/canvas/evas_object_textblock.c
+++ b/legacy/evas/src/lib/canvas/evas_object_textblock.c
@@ -5630,7 +5630,8 @@ evas_object_textblock_render_pre(Evas_Object *obj)
 	evas_object_render_pre_visible_change(&obj->layer->evas->clip_changes, obj, is_v, was_v);
 	goto done;
      }
-   if (obj->cur.map != obj->prev.map)
+   if ((obj->cur.map != obj->prev.map) ||
+       (obj->cur.usemap != obj->prev.usemap))
      {
 	evas_object_render_pre_prev_cur_add(&obj->layer->evas->clip_changes, obj);
         goto done;
diff --git a/legacy/evas/src/lib/canvas/evas_render.c b/legacy/evas/src/lib/canvas/evas_render.c
index 5ddb04134c..03d54ba2e5 100644
--- a/legacy/evas/src/lib/canvas/evas_render.c
+++ b/legacy/evas/src/lib/canvas/evas_render.c
@@ -85,54 +85,19 @@ evas_obscured_clear(Evas *e)
      }
 }
 
-static int
-_evas_child_changed_propagate(Evas_Object *obj)
-{
-   Evas_Object *obj2;
-   
-   EINA_INLIST_FOREACH(evas_object_smart_members_get_direct(obj), obj2)
-     {
-        if ((evas_object_is_visible(obj2) || 
-             evas_object_was_visible(obj2)))
-          {
-             if (((obj2->restack) && (!obj->clip.clipees)) ||
-                 (obj2->changed))
-               {
-                  obj->changed = 1;
-//                  return 1;
-               }
-             else if (obj2->smart.smart)
-               {
-                  if (obj2->restack) obj->changed = 1;
-                  obj->changed |= _evas_child_changed_propagate(obj2);
-/*                  
-                  if (obj2->changed)
-                    {
-                       obj->changed = 1;
-                       return obj->changed;
-                    }
-                  else if (_evas_child_changed_propagate(obj2))
-                    return obj->changed;
- */
-               }
-          }
-     }
-   return obj->changed;
-}
-
 static void
 _evas_render_phase1_direct(Evas *e, Eina_Array *active_objects, Eina_Array *restack_objects, Eina_Array *delete_objects, Eina_Array *render_objects)
 {
    unsigned int i;
 
-   for (i = 0; i < render_objects->count; ++i)
+   for (i = 0; i < render_objects->count; i++)
      {
 	Evas_Object *obj;
 
 	obj = eina_array_data_get(render_objects, i);
-        if (obj->smart.smart) _evas_child_changed_propagate(obj);
 	if (obj->changed)
           {
+             evas_object_clip_recalc(obj);
              obj->func->render_pre(obj);
              if (obj->pre_render_done)
                {
@@ -140,15 +105,37 @@ _evas_render_phase1_direct(Evas *e, Eina_Array *active_objects, Eina_Array *rest
                       (!((obj->func->can_map) && (obj->func->can_map(obj)))) &&
                       ((obj->cur.map) && (obj->cur.map->count == 4) && (obj->cur.usemap)))
                     {
-                       eina_array_push(restack_objects, obj);
+                       e->engine.func->output_redraws_rect_add(e->engine.data.output,
+                                                               obj->prev.cache.clip.x,
+                                                               obj->prev.cache.clip.y,
+                                                               obj->prev.cache.clip.w,
+                                                               obj->prev.cache.clip.h);
+                       e->engine.func->output_redraws_rect_add(e->engine.data.output,
+                                                               obj->cur.cache.clip.x,
+                                                               obj->cur.cache.clip.y,
+                                                               obj->cur.cache.clip.w,
+                                                               obj->cur.cache.clip.h);
                     }
                }
+             else if ((!obj->cur.map) && (obj->prev.usemap))
+               {
+                  e->engine.func->output_redraws_rect_add(e->engine.data.output,
+                                                          obj->prev.cache.clip.x,
+                                                          obj->prev.cache.clip.y,
+                                                          obj->prev.cache.clip.w,
+                                                          obj->prev.cache.clip.h);
+                  e->engine.func->output_redraws_rect_add(e->engine.data.output,
+                                                          obj->cur.cache.clip.x,
+                                                          obj->cur.cache.clip.y,
+                                                          obj->cur.cache.clip.w,
+                                                          obj->cur.cache.clip.h);
+               }
           }
 	else
 	  {
 	     if (obj->smart.smart)
                {
-                  obj->func->render_pre(obj);
+//                  obj->func->render_pre(obj);
                }
 	     else if (obj->rect_del)
                {
@@ -171,8 +158,6 @@ _evas_render_phase1_object_process(Evas *e, Evas_Object *obj, Eina_Array *active
    obj->rect_del = 0;
    obj->render_pre = 0;
 
-/* if (obj->cur.cache.clip.dirty) */
-   evas_object_clip_recalc(obj);
    /* because of clip objects - delete 2 cycles later */
    if (obj->delete_me == 2)
      eina_array_push(delete_objects, obj);
@@ -192,25 +177,35 @@ _evas_render_phase1_object_process(Evas *e, Evas_Object *obj, Eina_Array *active
    if (restack)
      {
 	if (!obj->changed)
-	  eina_array_push(&e->pending_objects, obj);
+          {
+             eina_array_push(&e->pending_objects, obj);
+             obj->changed = 1;
+          }
 	obj->restack = 1;
-	obj->changed = 1;
 	clean_them = EINA_TRUE;
      }
 
-   if (obj->smart.smart) _evas_child_changed_propagate(obj);
-
    if (!((obj->func->can_map) && (obj->func->can_map(obj))) &&
        ((obj->cur.map) && (obj->cur.map->count == 4) && (obj->cur.usemap)))
      {
         if (obj->changed)
           {
+             evas_object_clip_recalc(obj);
              if ((is_active) && (obj->restack) && (!obj->clip.clipees) &&
                  ((evas_object_is_visible(obj) && (!obj->cur.have_clipees)) ||
                   (evas_object_was_visible(obj) && (!obj->prev.have_clipees))))
                {
                   eina_array_push(render_objects, obj);
-                  eina_array_push(restack_objects, obj);
+                  e->engine.func->output_redraws_rect_add(e->engine.data.output,
+                                                          obj->prev.cache.clip.x,
+                                                          obj->prev.cache.clip.y,
+                                                          obj->prev.cache.clip.w,
+                                                          obj->prev.cache.clip.h);
+                  e->engine.func->output_redraws_rect_add(e->engine.data.output,
+                                                          obj->cur.cache.clip.x,
+                                                          obj->cur.cache.clip.y,
+                                                          obj->cur.cache.clip.w,
+                                                          obj->cur.cache.clip.h);
                   obj->render_pre = 1;
                }
              else if ((is_active) && (!obj->clip.clipees) &&
@@ -218,17 +213,42 @@ _evas_render_phase1_object_process(Evas *e, Evas_Object *obj, Eina_Array *active
                        (evas_object_was_visible(obj) && (!obj->prev.have_clipees))))
                {
                   eina_array_push(render_objects, obj);
-                  eina_array_push(restack_objects, obj);
+                  e->engine.func->output_redraws_rect_add(e->engine.data.output,
+                                                          obj->prev.cache.clip.x,
+                                                          obj->prev.cache.clip.y,
+                                                          obj->prev.cache.clip.w,
+                                                          obj->prev.cache.clip.h);
+                  e->engine.func->output_redraws_rect_add(e->engine.data.output,
+                                                          obj->cur.cache.clip.x,
+                                                          obj->cur.cache.clip.y,
+                                                          obj->cur.cache.clip.w,
+                                                          obj->cur.cache.clip.h);
                   obj->render_pre = 1;
                }
           }
         return clean_them;
      }
+   else if ((!obj->cur.usemap) && (obj->prev.usemap))
+     {
+        evas_object_clip_recalc(obj);
+        eina_array_push(restack_objects, obj);
+        e->engine.func->output_redraws_rect_add(e->engine.data.output,
+                                                obj->prev.cache.clip.x,
+                                                obj->prev.cache.clip.y,
+                                                obj->prev.cache.clip.w,
+                                                obj->prev.cache.clip.h);
+        e->engine.func->output_redraws_rect_add(e->engine.data.output,
+                                                obj->cur.cache.clip.x,
+                                                obj->cur.cache.clip.y,
+                                                obj->cur.cache.clip.w,
+                                                obj->cur.cache.clip.h);
+     }
    
    /* handle normal rendering. this object knows how to handle maps */
    
    if (obj->changed)
      {
+        evas_object_clip_recalc(obj);
 	if (obj->smart.smart)
 	  {
 	     Evas_Object *obj2;
@@ -397,7 +417,8 @@ pending_change(void *data, void *gdata __UNUSED__)
    obj = data;
    if (obj->delete_me) return EINA_FALSE;
    obj->pre_render_done = 0;
-   if (!obj->layer) obj->changed = 0;
+//   if (!obj->layer)
+     obj->changed = 0;
    return obj->changed ? EINA_TRUE : EINA_FALSE;
 }
 
@@ -407,6 +428,18 @@ evas_render_mapped(Evas *e, Evas_Object *obj, void *context, void *surface,
 {
    void *ctx;
    
+   evas_object_clip_recalc(obj);
+   if (mapped)
+     {
+        if ((!evas_object_is_visible(obj)) || (obj->clip.clipees) ||
+            (obj->cur.have_clipees))
+          return;
+     }
+   else if (!(((evas_object_is_active(obj) && (!obj->clip.clipees) &&
+                ((evas_object_is_visible(obj) && (!obj->cur.have_clipees)))))
+              )) 
+     return;
+   
    if (!((obj->func->can_map) && (obj->func->can_map(obj))) &&
        ((obj->cur.map) && (obj->cur.map->count == 4) && (obj->cur.usemap)))
      {
@@ -462,6 +495,7 @@ evas_render_mapped(Evas *e, Evas_Object *obj, void *context, void *surface,
                {
                   if (obj2->changed)
                     {
+                       obj2->changed  = 0;
                        changed = 1;
                        break;
                     }
@@ -469,11 +503,16 @@ evas_render_mapped(Evas *e, Evas_Object *obj, void *context, void *surface,
              obj->changed = 0;
           }
         else
-          changed = obj->changed;
+          {
+             changed = obj->changed;
+             obj->changed = 0;
+          }
         
         // clear surface before re-render
         if ((changed) && (obj->cur.map->surface))
           {
+             // FIXME: calculate "changes" within map surface and only clear
+             // and re-render those
              if (obj->cur.map->alpha)
                {
                   ctx = e->engine.func->context_new(e->engine.data.output);
@@ -499,15 +538,9 @@ evas_render_mapped(Evas *e, Evas_Object *obj, void *context, void *surface,
                   EINA_INLIST_FOREACH
                     (evas_object_smart_members_get_direct(obj), obj2)
                     {
-                       if (evas_object_is_active(obj2) && 
-                           (!obj2->clip.clipees) &&
-                           ((evas_object_is_visible(obj2) && 
-                             (!obj2->cur.have_clipees))))
-                         {
-                            evas_render_mapped(e, obj2, ctx, 
-                                               obj->cur.map->surface, 
-                                               off_x, off_y, 1);
-                         }
+                       evas_render_mapped(e, obj2, ctx, 
+                                          obj->cur.map->surface, 
+                                          off_x, off_y, 1);
                     }
                }
              else
@@ -539,34 +572,24 @@ evas_render_mapped(Evas *e, Evas_Object *obj, void *context, void *surface,
                {
                   Evas_Object *obj2;
                   
-//                  off_x = -obj->cur.geometry.x;
-//                  off_y = -obj->cur.geometry.y;
                   EINA_INLIST_FOREACH
                     (evas_object_smart_members_get_direct(obj), obj2)
                     {
-                       if (evas_object_is_active(obj2) &&
-                           (!obj2->clip.clipees) &&
-                           ((evas_object_is_visible(obj2) &&
-                             (!obj2->cur.have_clipees))))
-                         {
-                            evas_render_mapped(e, obj2, ctx,
-                                               surface,
-                                               off_x, off_y, 1);
-                         }
+                       evas_render_mapped(e, obj2, ctx,
+                                          surface,
+                                          off_x, off_y, 1);
                     }
                }
              else
-               {
-//                  off_x = -obj->cur.geometry.x;
-//                  off_y = -obj->cur.geometry.y;
-                  obj->func->render(obj, e->engine.data.output, ctx,
-                                    surface, off_x, off_y);
-               }
+               obj->func->render(obj, e->engine.data.output, ctx,
+                                 surface, off_x, off_y);
              e->engine.func->context_free(e->engine.data.output, ctx);
           }
         else
-          obj->func->render(obj, e->engine.data.output, context, surface, 
-                            off_x, off_y);
+          {
+             obj->func->render(obj, e->engine.data.output, context, surface, 
+                               off_x, off_y);
+          }
      }
 }
 
@@ -591,7 +614,7 @@ evas_render_updates_internal(Evas *e,
    if (!e->changed) return NULL;
 
    evas_call_smarts_calculate(e);
-
+   
    /* Check if the modified object mean recalculating every thing */
    if (!e->invalidate)
      _evas_render_check_pending_objects(&e->pending_objects, e);
@@ -759,7 +782,6 @@ evas_render_updates_internal(Evas *e,
 		  e->engine.func->context_cutout_clear(e->engine.data.output,
 						       e->engine.data.context);
 	       }
-
 	     /* render all object that intersect with rect */
              for (i = 0; i < e->active_objects.count; ++i)
 	       {
@@ -863,7 +885,8 @@ evas_render_updates_internal(Evas *e,
 	     obj->restack = 0;
 	     obj->changed = 0;
 	  }
-        else if (obj->cur.map != obj->prev.map)
+        else if ((obj->cur.map != obj->prev.map) ||
+                 (obj->cur.usemap != obj->prev.usemap))
           {
 	     obj->func->render_post(obj);
 	     obj->restack = 0;
@@ -907,12 +930,16 @@ evas_render_updates_internal(Evas *e,
         obj->pre_render_done = 0;
      }
    
-   /* If their are some object to restack or some object to delete, it's useless to keep the render object list around. */
-   if (clean_them)
+   /* If their are some object to restack or some object to delete, 
+    * it's useless to keep the render object list around. */
+//   if (clean_them)
+   if (1)
      {
 	eina_array_clean(&e->active_objects);
 	eina_array_clean(&e->render_objects);
-
+	eina_array_clean(&e->restack_objects);
+	eina_array_clean(&e->delete_objects);
+        eina_array_clean(&e->obscuring_objects);
 	e->invalidate = 1;
      }
 
@@ -1062,9 +1089,7 @@ evas_render_object_recalc(Evas_Object *obj)
 
 	e = obj->layer->evas;
 	if ((!e) || (e->cleanup)) return;
-
-	if (!obj->changed)
-	  eina_array_push(&e->pending_objects, obj);
+        eina_array_push(&e->pending_objects, obj);
 	obj->changed = 1;
      }
 }
diff --git a/legacy/evas/src/lib/include/evas_inline.x b/legacy/evas/src/lib/include/evas_inline.x
index 5bf29650e8..7cc135ef32 100644
--- a/legacy/evas/src/lib/include/evas_inline.x
+++ b/legacy/evas/src/lib/include/evas_inline.x
@@ -154,8 +154,9 @@ evas_object_clip_recalc(Evas_Object *obj)
    int nx, ny, nw, nh, nvis, nr, ng, nb, na;
 
    if (obj->layer->evas->events_frozen > 0) return;
-   if (!(obj->cur.clipper == NULL || obj->cur.clipper->cur.cache.clip.dirty)
-       && !obj->cur.cache.clip.dirty) return;
+   if (!obj->cur.cache.clip.dirty &&
+       !(obj->cur.clipper == NULL || obj->cur.clipper->cur.cache.clip.dirty))
+     return;
    evas_object_coords_recalc(obj);
    if (obj->cur.map)
      {