3d, gesturelayer, index, naviframe, transit, fileselector, frame, glview, layout...
[framework/uifw/elementary.git] / src / lib / elm_transit.c
index f050b21..3116daa 100644 (file)
@@ -28,12 +28,11 @@ struct _Elm_Transit
    Ecore_Animator *animator;
    Eina_Inlist *effect_list;
    Eina_List *objs;
-   Eina_Hash *objs_data_hash;
    Elm_Transit *prev_chain_transit;
    Eina_List *next_chain_transits;
    Elm_Transit_Tween_Mode tween_mode;
    struct {
-      Elm_Transit_Effect_End_Cb func;
+      Elm_Transit_Del_Cb func;
       void *arg;
    } del_data;
    struct {
@@ -55,6 +54,7 @@ struct _Elm_Transit
    Eina_Bool event_enabled : 1;
    Eina_Bool deleted : 1;
    Eina_Bool state_keep : 1;
+   Eina_Bool finished : 1;
 };
 
 struct _Elm_Transit_Effect_Module
@@ -66,7 +66,7 @@ struct _Elm_Transit_Effect_Module
    Eina_Bool deleted : 1;
 };
 
-struct _Elm_Obj_State
+struct _Elm_Transit_Obj_State
 {
    Evas_Coord x, y, w, h;
    int r,g,b,a;
@@ -75,113 +75,74 @@ struct _Elm_Obj_State
    Eina_Bool visible : 1;
 };
 
-struct _Elm_Obj_Data
+struct _Elm_Transit_Obj_Data
 {
-   struct _Elm_Obj_State *state;
-   Eina_Bool pass_events : 1;
+   struct _Elm_Transit_Obj_State *state;
+   Eina_Bool freeze_events : 1;
 };
 
 typedef struct _Elm_Transit_Effect_Module Elm_Transit_Effect_Module;
-typedef struct _Elm_Obj_Data Elm_Obj_Data;
-typedef struct _Elm_Obj_State Elm_Obj_State;
+typedef struct _Elm_Transit_Obj_Data Elm_Transit_Obj_Data;
+typedef struct _Elm_Transit_Obj_State Elm_Transit_Obj_State;
+
+static void _transit_obj_data_update(Elm_Transit *transit, Evas_Object *obj);
+static void _transit_obj_data_recover(Elm_Transit *transit, Evas_Object *obj);
+static void _transit_obj_states_save(Evas_Object *obj, Elm_Transit_Obj_Data *obj_data);
+static void _transit_obj_remove_cb(void *data, Evas *e __UNUSED__, Evas_Object *obj, void *event_info __UNUSED__);
+static void _transit_obj_remove(Elm_Transit *transit, Evas_Object *obj);
+static void _transit_effect_del(Elm_Transit *transit, Elm_Transit_Effect_Module *effect_module);
+static void _transit_remove_dead_effects(Elm_Transit *transit);
+static void _transit_chain_go(Elm_Transit *transit);
+static void _transit_del(Elm_Transit *transit);
+static Eina_Bool _transit_animate_op(Elm_Transit *transit, double progress);
+static Eina_Bool _transit_animate_cb(void *data);
+
+static char *_transit_key= "_elm_transit_key";
 
 static void
-_elm_transit_obj_states_save(Evas_Object *obj, Elm_Obj_Data *obj_data)
+_transit_obj_data_update(Elm_Transit *transit, Evas_Object *obj)
 {
-   Elm_Obj_State *state;
+   Elm_Transit_Obj_Data *obj_data = evas_object_data_get(obj, _transit_key);
 
-   if (obj_data->state) return;
-   state = calloc(1, sizeof(Elm_Obj_State));
-   if (!state) return;
-   evas_object_geometry_get(obj, &state->x, &state->y, &state->w, &state->h);
-   evas_object_color_get(obj, &state->r, &state->g, &state->b, &state->a);
-   state->visible = evas_object_visible_get(obj);
-   state->map_enabled = evas_object_map_enable_get(obj);
-   if (evas_object_map_get(obj))
-     state->map = evas_map_dup(evas_object_map_get(obj));
-   obj_data->state = state;
-}
-
-static Eina_Bool
-_hash_foreach_pass_events_set(const Eina_Hash *hash __UNUSED__, const void *key, void *data __UNUSED__, void *fdata)
-{
-   Elm_Transit *transit = fdata;
-   evas_object_pass_events_set((Evas_Object*) key, transit->event_enabled);
-   return EINA_TRUE;
-}
+   if (!obj_data)
+     obj_data = ELM_NEW(Elm_Transit_Obj_Data);
 
-static Eina_Bool
-_hash_foreach_obj_states_save(const Eina_Hash *hash __UNUSED__, const void *key, void *data, void *fdata __UNUSED__)
-{
-   _elm_transit_obj_states_save((Evas_Object *) key, (Elm_Obj_Data *) data);
-   return EINA_TRUE;
-}
+   obj_data->freeze_events = evas_object_freeze_events_get(obj);
 
-static void
-_elm_transit_object_remove_cb(void *data, Evas *e __UNUSED__, Evas_Object *obj, void *event_info __UNUSED__)
-{
-   Elm_Transit *transit;
-   Elm_Obj_Data *obj_data;
-   Eina_List *list;
+   if ((!transit->state_keep) && (obj_data->state))
+     {
+        free(obj_data->state);
+        obj_data->state = NULL;
+     }
+   else
+     {
+       _transit_obj_states_save(obj, obj_data);
+     }
 
-   transit = data;
-   list = eina_list_data_find_list(transit->objs, obj);
-   obj_data = eina_hash_find(transit->objs_data_hash, list);
-   if (!obj_data) return;
-   eina_hash_del_by_key(transit->objs_data_hash, list);
-   evas_object_pass_events_set(obj, obj_data->pass_events);
-   if (obj_data->state)
-     free(obj_data->state);
-   free(obj_data);
-   transit->objs = eina_list_remove(transit->objs, obj);
-   if (!transit->objs) elm_transit_del(transit);
+   evas_object_data_set(obj, _transit_key, obj_data);
 }
 
-//TODO: Remove!
-//Since evas map have a afterimage bug for this time.
-//This function is added temporary.
 static void
-_obj_damage_area_set(Evas_Object *obj)
+_transit_obj_states_save(Evas_Object *obj, Elm_Transit_Obj_Data *obj_data)
 {
-   const Evas_Map *map;
-   Evas_Coord_Point coords;
-   Evas_Coord_Point min, max;
-   int i;
-
-   map  = evas_object_map_get(obj);
-   if (!map) return;
-
-   evas_map_point_coord_get(map, 0, &coords.x, &coords.y, NULL);
+   Elm_Transit_Obj_State *state = obj_data->state;
 
-   max = min = coords;
-
-   for (i = 1; i < 4; ++i)
-     {
-        evas_map_point_coord_get(map, i, &coords.x, &coords.y, NULL);
-
-        if (coords.x < min.x)
-          min.x = coords.x;
-        else if (coords.x > max.x)
-          max.x = coords.x;
-
-        if (coords.y < min.y)
-          min.y = coords.y;
-        else if (coords.y > max.y)
-          max.y = coords.y;
-     }
+   if (!state)
+     state = calloc(1, sizeof(Elm_Transit_Obj_State));
+   if (!state) return;
 
-   evas_damage_rectangle_add(evas_object_evas_get(obj),
-                             min.x, min.y,
-                             max.x - min.x, max.y - min.y);
+   evas_object_geometry_get(obj, &state->x, &state->y, &state->w, &state->h);
+   evas_object_color_get(obj, &state->r, &state->g, &state->b, &state->a);
+   state->visible = evas_object_visible_get(obj);
+   state->map_enabled = evas_object_map_enable_get(obj);
+   if (evas_object_map_get(obj))
+     state->map = evas_map_dup(evas_object_map_get(obj));
+   obj_data->state = state;
 }
 
 static void
 _remove_obj_from_list(Elm_Transit *transit, Evas_Object *obj)
 {
-   evas_object_event_callback_del_full(obj, EVAS_CALLBACK_DEL,
-                                       _elm_transit_object_remove_cb,
-                                       transit);
-
    //Remove duplicated objects
    //TODO: Need to consider about optimizing here
    while(1)
@@ -189,26 +150,37 @@ _remove_obj_from_list(Elm_Transit *transit, Evas_Object *obj)
         if (!eina_list_data_find_list(transit->objs, obj))
           break;
         transit->objs = eina_list_remove(transit->objs, obj);
+        evas_object_event_callback_del_full(obj, EVAS_CALLBACK_DEL,
+                                       _transit_obj_remove_cb,
+                                       transit);
      }
 }
 
 static void
-_elm_transit_object_remove(Elm_Transit *transit, Evas_Object *obj)
+_transit_obj_remove_cb(void *data, Evas *e __UNUSED__, Evas_Object *obj, void *event_info __UNUSED__)
 {
-   Elm_Obj_Data *obj_data;
-   Elm_Obj_State *state;
-   Eina_List *list;
-
-   list = eina_list_data_find_list(transit->objs, obj);
-   obj_data = eina_hash_find(transit->objs_data_hash, list);
-   if (!obj_data)
+   Elm_Transit *transit = data;
+   Elm_Transit_Obj_Data *obj_data = evas_object_data_get(obj, _transit_key);
+   if (obj_data)
      {
-        _remove_obj_from_list(transit, obj);
-        return;
+        if (obj_data->state)
+          free(obj_data->state);
+        free(obj_data);
      }
-   eina_hash_del_by_key(transit->objs_data_hash, list);
    _remove_obj_from_list(transit, obj);
-   evas_object_pass_events_set(obj, obj_data->pass_events);
+   if (!transit->objs) elm_transit_del(transit);
+}
+
+static void
+_transit_obj_data_recover(Elm_Transit *transit, Evas_Object *obj)
+{
+   Elm_Transit_Obj_Data *obj_data;
+   Elm_Transit_Obj_State *state;
+
+   obj_data = evas_object_data_get(obj, _transit_key);
+   if (!obj_data) return;
+   evas_object_data_del(obj, _transit_key);
+   evas_object_freeze_events_set(obj, obj_data->freeze_events);
    state = obj_data->state;
    if (state)
      {
@@ -226,21 +198,21 @@ _elm_transit_object_remove(Elm_Transit *transit, Evas_Object *obj)
                evas_object_map_enable_set(obj, EINA_FALSE);
              if (state->map)
                evas_object_map_set(obj, state->map);
-
-             //TODO: Remove!
-             //Since evas map have a afterimage bug for this time.
-             //This line is added temporary.
-             _obj_damage_area_set(obj);
-
           }
         free(state);
      }
    free(obj_data);
+}
 
+static void
+_transit_obj_remove(Elm_Transit *transit, Evas_Object *obj)
+{
+   _remove_obj_from_list(transit, obj);
+   _transit_obj_data_recover(transit, obj);
 }
 
 static void
-_elm_transit_effect_del(Elm_Transit *transit, Elm_Transit_Effect_Module *effect_module)
+_transit_effect_del(Elm_Transit *transit, Elm_Transit_Effect_Module *effect_module)
 {
    if (effect_module->end_cb)
      effect_module->end_cb(effect_module->effect, transit);
@@ -248,7 +220,7 @@ _elm_transit_effect_del(Elm_Transit *transit, Elm_Transit_Effect_Module *effect_
 }
 
 static void
-_remove_dead_effects(Elm_Transit *transit)
+_transit_remove_dead_effects(Elm_Transit *transit)
 {
    Elm_Transit_Effect_Module *effect_module;
 
@@ -256,7 +228,7 @@ _remove_dead_effects(Elm_Transit *transit)
      {
         if (effect_module->deleted)
           {
-             _elm_transit_effect_del(transit, effect_module);
+             _transit_effect_del(transit, effect_module);
              transit->effects_pending_del--;
              if (!transit->effects_pending_del) return;
           }
@@ -264,56 +236,64 @@ _remove_dead_effects(Elm_Transit *transit)
 }
 
 static void
-_elm_transit_del(Elm_Transit *transit)
+_transit_chain_go(Elm_Transit *transit)
+{
+   ELM_TRANSIT_CHECK_OR_RETURN(transit);
+   elm_transit_go(transit);
+   _transit_animate_cb(transit);
+}
+
+static void
+_transit_del(Elm_Transit *transit)
 {
    Elm_Transit_Effect_Module *effect_module;
    Elm_Transit *chain_transit;
    Eina_List *elist, *elist_next;
 
-   EINA_LIST_FOREACH_SAFE(transit->next_chain_transits, elist, elist_next, chain_transit)
-     {
-        if (transit->prev_chain_transit)
-          transit->prev_chain_transit->next_chain_transits = eina_list_remove(transit->prev_chain_transit->next_chain_transits, transit);
-        chain_transit->prev_chain_transit = NULL;
-     }
-
-   eina_list_free(transit->next_chain_transits);
-
    if (transit->animator)
      ecore_animator_del(transit->animator);
 
+   //remove effects
    while (transit->effect_list)
      {
         effect_module = EINA_INLIST_CONTAINER_GET(transit->effect_list, Elm_Transit_Effect_Module);
         transit->effect_list = eina_inlist_remove(transit->effect_list, transit->effect_list);
-        _elm_transit_effect_del(transit, effect_module);
+        _transit_effect_del(transit, effect_module);
      }
 
+   //remove objects.
    while (transit->objs)
-     _elm_transit_object_remove(transit, eina_list_data_get(transit->objs));
+     _transit_obj_remove(transit, eina_list_data_get(transit->objs));
 
    transit->deleted = EINA_TRUE;
 
    if (transit->del_data.func)
      transit->del_data.func(transit->del_data.arg, transit);
 
-   eina_hash_free(transit->objs_data_hash);
+   //cut off the chain transit relationship
+   EINA_LIST_FOREACH_SAFE(transit->next_chain_transits, elist, elist_next, chain_transit)
+     chain_transit->prev_chain_transit = NULL;
 
-   EINA_MAGIC_SET(transit, EINA_MAGIC_NONE);
-   free(transit);
-}
+   if (transit->prev_chain_transit)
+     transit->prev_chain_transit->next_chain_transits =
+        eina_list_remove(transit->prev_chain_transit->next_chain_transits, transit);
 
-static void
-_chain_transits_go(Elm_Transit *transit)
-{
-   Eina_List *elist, *elist_next;
-   Elm_Transit *chain_transit;
+   // run chain transits
+   if (transit->finished && transit->next_chain_transits)
+     {
+        EINA_LIST_FOREACH_SAFE(transit->next_chain_transits, elist, elist_next, chain_transit)
+          _transit_chain_go(chain_transit);
 
-   EINA_LIST_FOREACH_SAFE(transit->next_chain_transits, elist, elist_next, chain_transit)
-     elm_transit_go(chain_transit);
+     }
+
+   eina_list_free(transit->next_chain_transits);
+
+   EINA_MAGIC_SET(transit, EINA_MAGIC_NONE);
+   free(transit);
 }
 
-static void
+//If the transit is deleted then EINA_FALSE is retruned.
+static Eina_Bool
 _transit_animate_op(Elm_Transit *transit, double progress)
 {
    Elm_Transit_Effect_Module *effect_module;
@@ -327,14 +307,21 @@ _transit_animate_op(Elm_Transit *transit, double progress)
      }
    transit->walking--;
 
-   if (transit->walking) return;
+   if (transit->walking) return EINA_TRUE;
+
+   if (transit->deleted)
+     {
+        _transit_del(transit);
+        return EINA_FALSE;
+     }
 
-   if (transit->deleted) _elm_transit_del(transit);
-   else if (transit->effects_pending_del) _remove_dead_effects(transit);
+   else if (transit->effects_pending_del) _transit_remove_dead_effects(transit);
+
+   return EINA_TRUE;
 }
 
 static Eina_Bool
-_animator_animate_cb(void *data)
+_transit_animate_cb(void *data)
 {
    Elm_Transit *transit = data;
    double elapsed_time, duration;
@@ -349,23 +336,37 @@ _animator_animate_cb(void *data)
    transit->progress = elapsed_time / duration;
    switch (transit->tween_mode)
      {
+      case ELM_TRANSIT_TWEEN_MODE_LINEAR:
+         transit->progress = ecore_animator_pos_map(transit->progress,
+                                                    ECORE_POS_MAP_LINEAR,
+                                                    0, 0);
       case ELM_TRANSIT_TWEEN_MODE_ACCELERATE:
-        transit->progress = 1.0 - sin((ELM_PI / 2.0) + (transit->progress * ELM_PI / 2.0));
-        break;
+         transit->progress = ecore_animator_pos_map(transit->progress,
+                                                    ECORE_POS_MAP_ACCELERATE,
+                                                    0, 0);
+         break;
       case ELM_TRANSIT_TWEEN_MODE_DECELERATE:
-        transit->progress = sin(transit->progress * ELM_PI / 2.0);
-        break;
+         transit->progress = ecore_animator_pos_map(transit->progress,
+                                                    ECORE_POS_MAP_DECELERATE,
+                                                    0, 0);
+         break;
       case ELM_TRANSIT_TWEEN_MODE_SINUSOIDAL:
-        transit->progress = (1.0 - cos(transit->progress * ELM_PI)) / 2.0;
-        break;
+         transit->progress = ecore_animator_pos_map(transit->progress,
+                                                    ECORE_POS_MAP_SINUSOIDAL,
+                                                    0, 0);
+         break;
       default:
-        break;
+         break;
      }
 
    /* Reverse? */
    if (transit->repeat.reverse) transit->progress = 1 - transit->progress;
 
-   if (transit->time.duration > 0) _transit_animate_op(transit, transit->progress);
+   if (transit->time.duration > 0)
+     {
+        if (!_transit_animate_op(transit, transit->progress))
+          return ECORE_CALLBACK_CANCEL;
+     }
 
    /* Not end. Keep going. */
    if (elapsed_time < duration) return ECORE_CALLBACK_RENEW;
@@ -375,10 +376,7 @@ _animator_animate_cb(void *data)
        (transit->repeat.current == transit->repeat.count) &&
        ((!transit->auto_reverse) || transit->repeat.reverse))
      {
-        /* run chain transit */
-        if (transit->next_chain_transits)
-          _chain_transits_go(transit);
-
+        transit->finished = EINA_TRUE;
         elm_transit_del(transit);
         return ECORE_CALLBACK_CANCEL;
      }
@@ -400,14 +398,16 @@ EAPI Elm_Transit *
 elm_transit_add(void)
 {
    Elm_Transit *transit = ELM_NEW(Elm_Transit);
-   if (!transit) return NULL;
+   if (!transit)
+     {
+        ERR("Failed to allocate a elm_transit object!");
+        return NULL;
+     }
 
    EINA_MAGIC_SET(transit, ELM_TRANSIT_MAGIC);
 
    elm_transit_tween_mode_set(transit, ELM_TRANSIT_TWEEN_MODE_LINEAR);
 
-   transit->objs_data_hash = eina_hash_int32_new(NULL);
-
    return transit;
 }
 
@@ -417,7 +417,7 @@ elm_transit_del(Elm_Transit *transit)
    ELM_TRANSIT_CHECK_OR_RETURN(transit);
 
    if (transit->walking) transit->deleted = EINA_TRUE;
-   else _elm_transit_del(transit);
+   else _transit_del(transit);
 }
 
 EAPI void
@@ -428,10 +428,18 @@ elm_transit_effect_add(Elm_Transit *transit, Elm_Transit_Effect_Transition_Cb tr
    Elm_Transit_Effect_Module *effect_module;
 
    EINA_INLIST_FOREACH(transit->effect_list, effect_module)
-     if ((effect_module->transition_cb == transition_cb) && (effect_module->effect == effect)) return;
+     if ((effect_module->transition_cb == transition_cb) && (effect_module->effect == effect))
+       {
+          WRN("elm_transit does not allow to add the duplicated effect! : transit=%p", transit);
+          return;
+       }
 
    effect_module = ELM_NEW(Elm_Transit_Effect_Module);
-   if (!effect_module) return;
+   if (!effect_module)
+     {
+        ERR("Failed to allocate a new effect!: transit=%p", transit);
+        return;
+     }
 
    effect_module->end_cb = end_cb;
    effect_module->transition_cb = transition_cb;
@@ -458,7 +466,7 @@ elm_transit_effect_del(Elm_Transit *transit, Elm_Transit_Effect_Transition_Cb tr
                }
              else
                {
-                  _elm_transit_effect_del(transit, effect_module);
+                  _transit_effect_del(transit, effect_module);
                   if (!transit->effect_list) elm_transit_del(transit);
                }
              return;
@@ -471,25 +479,21 @@ elm_transit_object_add(Elm_Transit *transit, Evas_Object *obj)
 {
    ELM_TRANSIT_CHECK_OR_RETURN(transit);
    EINA_SAFETY_ON_NULL_RETURN(obj);
-   Elm_Obj_Data *obj_data;
-   Eina_List * list;
 
-//TODO: Check the remove case of the same objects in this transit.
-   obj_data = ELM_NEW(Elm_Obj_Data);
-   obj_data->pass_events = evas_object_pass_events_get(obj);
-   if (!transit->event_enabled)
-     evas_object_pass_events_set(obj, EINA_TRUE);
+   if (transit->animator)
+     {
+        if (!evas_object_data_get(obj, _transit_key))
+          {
+             _transit_obj_data_update(transit, obj);
+             evas_object_freeze_events_set(obj, EINA_TRUE);
+          }
+     }
 
    evas_object_event_callback_add(obj, EVAS_CALLBACK_DEL,
-                                  _elm_transit_object_remove_cb,
+                                  _transit_obj_remove_cb,
                                   transit);
 
    transit->objs = eina_list_append(transit->objs, obj);
-   list = eina_list_last(transit->objs);
-   eina_hash_add(transit->objs_data_hash, list, obj_data);
-
-   if (!transit->state_keep)
-     _elm_transit_obj_states_save(obj, obj_data);
 }
 
 EAPI void
@@ -498,7 +502,7 @@ elm_transit_object_remove(Elm_Transit *transit, Evas_Object *obj)
    ELM_TRANSIT_CHECK_OR_RETURN(transit);
    EINA_SAFETY_ON_NULL_RETURN(obj);
 
-   _elm_transit_object_remove(transit, obj);
+   _transit_obj_remove(transit, obj);
    if (!transit->objs) elm_transit_del(transit);
 }
 
@@ -514,9 +518,15 @@ elm_transit_event_enabled_set(Elm_Transit *transit, Eina_Bool enabled)
 {
    ELM_TRANSIT_CHECK_OR_RETURN(transit);
 
+   Eina_List *list;
+   Evas_Object *obj;
+
    if (transit->event_enabled == enabled) return;
    transit->event_enabled = !!enabled;
-   eina_hash_foreach(transit->objs_data_hash, _hash_foreach_pass_events_set, transit);
+   if (!transit->animator) return;
+
+   EINA_LIST_FOREACH(transit->objs, list, obj)
+     evas_object_freeze_events_set(obj, enabled);
 }
 
 EAPI Eina_Bool
@@ -526,20 +536,6 @@ elm_transit_event_enabled_get(const Elm_Transit *transit)
    return transit->event_enabled;
 }
 
-
-EINA_DEPRECATED EAPI void
-elm_transit_event_block_set(Elm_Transit *transit, Eina_Bool disabled)
-{
-   elm_transit_event_enabled_set(transit, disabled);
-}
-
-
-EINA_DEPRECATED EAPI Eina_Bool
-elm_transit_event_block_get(const Elm_Transit *transit)
-{
-   return !elm_transit_event_enabled_get(transit);
-}
-
 EAPI void
 elm_transit_del_cb_set(Elm_Transit *transit, void (*cb) (void *data, Elm_Transit *transit), void *data)
 {
@@ -595,7 +591,11 @@ EAPI void
 elm_transit_duration_set(Elm_Transit *transit, double duration)
 {
    ELM_TRANSIT_CHECK_OR_RETURN(transit);
-   if (transit->animator) return;
+   if (transit->animator)
+     {
+        WRN("elm_transit does not allow to set the duration time in operating! : transit=%p", transit);
+        return;
+     }
    transit->time.duration = duration;
 }
 
@@ -611,13 +611,25 @@ elm_transit_go(Elm_Transit *transit)
 {
    ELM_TRANSIT_CHECK_OR_RETURN(transit);
 
+   Eina_List *elist;
+   Evas_Object *obj;
+
    if (transit->animator)
      ecore_animator_del(transit->animator);
 
+   EINA_LIST_FOREACH(transit->objs, elist, obj)
+     _transit_obj_data_update(transit, obj);
+
+   if (!transit->event_enabled)
+     {
+        EINA_LIST_FOREACH(transit->objs, elist, obj)
+          evas_object_freeze_events_set(obj, EINA_TRUE);
+     }
+
    transit->time.paused = 0;
    transit->time.delayed = 0;
    transit->time.begin = ecore_loop_time_get();
-   transit->animator = ecore_animator_add(_animator_animate_cb, transit);
+   transit->animator = ecore_animator_add(_transit_animate_cb, transit);
 }
 
 EAPI void
@@ -659,6 +671,7 @@ EAPI double
 elm_transit_progress_value_get(const Elm_Transit *transit)
 {
    ELM_TRANSIT_CHECK_OR_RETURN(transit, 0);
+
    return transit->progress;
 }
 
@@ -666,11 +679,14 @@ EAPI void
 elm_transit_objects_final_state_keep_set(Elm_Transit *transit, Eina_Bool state_keep)
 {
    ELM_TRANSIT_CHECK_OR_RETURN(transit);
+
    if (transit->state_keep == state_keep) return;
-   if (transit->animator) return;
+   if (transit->animator)
+     {
+        WRN("elm_transit does not allow to change final state keep mode in operating! : transit=%p", transit);
+        return;
+     }
    transit->state_keep = !!state_keep;
-   if (state_keep) return;
-   eina_hash_foreach(transit->objs_data_hash, _hash_foreach_obj_states_save, NULL);
 }
 
 EAPI Eina_Bool
@@ -686,8 +702,13 @@ elm_transit_chain_transit_add(Elm_Transit *transit, Elm_Transit *chain_transit)
    ELM_TRANSIT_CHECK_OR_RETURN(transit);
    ELM_TRANSIT_CHECK_OR_RETURN(chain_transit);
 
-   if (transit == chain_transit) return;
-   if (transit == chain_transit->prev_chain_transit) return;
+   if (transit == chain_transit)
+     {
+        WRN("You add a same transit as a chain transit! : transit=%p, chain_transit=%p", transit, chain_transit);
+        return;
+     }
+   if (transit == chain_transit->prev_chain_transit)
+     return;
 
    if (chain_transit->prev_chain_transit)
      chain_transit->prev_chain_transit->next_chain_transits = eina_list_remove(chain_transit->prev_chain_transit->next_chain_transits, chain_transit);
@@ -696,6 +717,22 @@ elm_transit_chain_transit_add(Elm_Transit *transit, Elm_Transit *chain_transit)
    transit->next_chain_transits = eina_list_append(transit->next_chain_transits, chain_transit);
 }
 
+EAPI void
+elm_transit_chain_transit_del(Elm_Transit *transit, Elm_Transit *chain_transit)
+{
+   ELM_TRANSIT_CHECK_OR_RETURN(transit);
+   ELM_TRANSIT_CHECK_OR_RETURN(chain_transit);
+
+   if (chain_transit->prev_chain_transit != transit)
+     {
+        WRN("A pair of transits does not have the chain relationship! : transit=%p, chain_transit=%p", transit, chain_transit);
+        return;
+     }
+
+   chain_transit->prev_chain_transit = NULL;
+   transit->next_chain_transits = eina_list_remove(transit->next_chain_transits, chain_transit);
+}
+
 EAPI Eina_List *
 elm_transit_chain_transits_get(const Elm_Transit * transit)
 {
@@ -703,9 +740,9 @@ elm_transit_chain_transits_get(const Elm_Transit * transit)
    return transit->next_chain_transits;
 }
 
-///////////////////////////////////////////////////////////////////////////////
+///////////////////////////////////////////////////////////////////////////
 //Resizing Effect
-///////////////////////////////////////////////////////////////////////////////
+///////////////////////////////////////////////////////////////////////////
 typedef struct _Elm_Transit_Effect_Resizing Elm_Transit_Effect_Resizing;
 
 struct _Elm_Transit_Effect_Resizing
@@ -761,16 +798,20 @@ elm_transit_effect_resizing_add(Elm_Transit *transit, Evas_Coord from_w, Evas_Co
    ELM_TRANSIT_CHECK_OR_RETURN(transit, NULL);
    Elm_Transit_Effect *effect = _transit_effect_resizing_context_new(from_w, from_h, to_w, to_h);
 
-   if (!effect) return NULL;
+   if (!effect)
+     {
+        ERR("Failed to allocate resizing effect! : transit=%p", transit);
+        return NULL;
+     }
    elm_transit_effect_add(transit,
                           _transit_effect_resizing_op, effect,
                           _transit_effect_resizing_context_free);
    return effect;
 }
 
-///////////////////////////////////////////////////////////////////////////////
+///////////////////////////////////////////////////////////////////////////
 //Translation Effect
-///////////////////////////////////////////////////////////////////////////////
+///////////////////////////////////////////////////////////////////////////
 typedef struct _Elm_Transit_Effect_Translation Elm_Transit_Effect_Translation;
 typedef struct _Elm_Transit_Effect_Translation_Node Elm_Transit_Effect_Translation_Node;
 
@@ -893,19 +934,22 @@ EAPI Elm_Transit_Effect *
 elm_transit_effect_translation_add(Elm_Transit *transit, Evas_Coord from_dx, Evas_Coord from_dy, Evas_Coord to_dx, Evas_Coord to_dy)
 {
    ELM_TRANSIT_CHECK_OR_RETURN(transit, NULL);
-   Elm_Transit_Effect *effect_context = _transit_effect_translation_context_new(from_dx, from_dy, to_dx, to_dy);
+   Elm_Transit_Effect *effect = _transit_effect_translation_context_new(from_dx, from_dy, to_dx, to_dy);
 
-   if (!effect_context) return NULL;
+   if (!effect)
+     {
+        ERR("Failed to allocate translation effect! : transit=%p", transit);
+        return NULL;
+     }
    elm_transit_effect_add(transit,
-                          _transit_effect_translation_op, effect_context,
+                          _transit_effect_translation_op, effect,
                           _transit_effect_translation_context_free);
-   return effect_context;
+   return effect;
 }
 
-
-///////////////////////////////////////////////////////////////////////////////
+///////////////////////////////////////////////////////////////////////////
 //Zoom Effect
-///////////////////////////////////////////////////////////////////////////////
+///////////////////////////////////////////////////////////////////////////
 typedef struct _Elm_Transit_Effect_Zoom Elm_Transit_Effect_Zoom;
 
 struct _Elm_Transit_Effect_Zoom
@@ -964,19 +1008,22 @@ EAPI Elm_Transit_Effect *
 elm_transit_effect_zoom_add(Elm_Transit *transit, float from_rate, float to_rate)
 {
    ELM_TRANSIT_CHECK_OR_RETURN(transit, NULL);
-   Elm_Transit_Effect *effect_context = _transit_effect_zoom_context_new(from_rate, to_rate);
+   Elm_Transit_Effect *effect = _transit_effect_zoom_context_new(from_rate, to_rate);
 
-   if (!effect_context) return NULL;
+   if (!effect)
+     {
+        ERR("Failed to allocate zoom effect! : transit=%p", transit);
+        return NULL;
+     }
    elm_transit_effect_add(transit,
-                          _transit_effect_zoom_op, effect_context,
+                          _transit_effect_zoom_op, effect,
                           _transit_effect_zoom_context_free);
-   return effect_context;
+   return effect;
 }
 
-
-///////////////////////////////////////////////////////////////////////////////
+///////////////////////////////////////////////////////////////////////////
 //Flip Effect
-///////////////////////////////////////////////////////////////////////////////
+///////////////////////////////////////////////////////////////////////////
 typedef struct _Elm_Transit_Effect_Flip Elm_Transit_Effect_Flip;
 
 struct _Elm_Transit_Effect_Flip
@@ -1106,18 +1153,22 @@ EAPI Elm_Transit_Effect *
 elm_transit_effect_flip_add(Elm_Transit *transit, Elm_Transit_Effect_Flip_Axis axis, Eina_Bool cw)
 {
    ELM_TRANSIT_CHECK_OR_RETURN(transit, NULL);
-   Elm_Transit_Effect *effect_context = _transit_effect_flip_context_new(axis, cw);
+   Elm_Transit_Effect *effect = _transit_effect_flip_context_new(axis, cw);
 
-   if (!effect_context) return NULL;
+   if (!effect)
+     {
+        ERR("Failed to allocate flip effect! : transit=%p", transit);
+        return NULL;
+     }
    elm_transit_effect_add(transit,
-                          _transit_effect_flip_op, effect_context,
+                          _transit_effect_flip_op, effect,
                           _transit_effect_flip_context_free);
-   return effect_context;
+   return effect;
 }
 
-///////////////////////////////////////////////////////////////////////////////
+///////////////////////////////////////////////////////////////////////////
 //ResizableFlip Effect
-///////////////////////////////////////////////////////////////////////////////
+///////////////////////////////////////////////////////////////////////////
 typedef struct _Elm_Transit_Effect_Resizable_Flip Elm_Transit_Effect_ResizableFlip;
 typedef struct _Elm_Transit_Effect_Resizable_Flip_Node Elm_Transit_Effect_ResizableFlip_Node;
 
@@ -1385,19 +1436,22 @@ EAPI Elm_Transit_Effect *
 elm_transit_effect_resizable_flip_add(Elm_Transit *transit, Elm_Transit_Effect_Flip_Axis axis, Eina_Bool cw)
 {
    ELM_TRANSIT_CHECK_OR_RETURN(transit, NULL);
-   Elm_Transit_Effect *effect_context = _transit_effect_resizable_flip_context_new(axis, cw);
+   Elm_Transit_Effect *effect = _transit_effect_resizable_flip_context_new(axis, cw);
 
-   if (!effect_context) return NULL;
+   if (!effect)
+     {
+        ERR("Failed to allocate resizable_flip effect! : transit=%p", transit);
+        return NULL;
+     }
    elm_transit_effect_add(transit,
-                          _transit_effect_resizable_flip_op, effect_context,
+                          _transit_effect_resizable_flip_op, effect,
                           _transit_effect_resizable_flip_context_free);
-   return effect_context;
+   return effect;
 }
 
-
-///////////////////////////////////////////////////////////////////////////////
+///////////////////////////////////////////////////////////////////////////
 //Wipe Effect
-///////////////////////////////////////////////////////////////////////////////
+///////////////////////////////////////////////////////////////////////////
 typedef struct _Elm_Transit_Effect_Wipe Elm_Transit_Effect_Wipe;
 
 struct _Elm_Transit_Effect_Wipe
@@ -1597,19 +1651,22 @@ EAPI void *
 elm_transit_effect_wipe_add(Elm_Transit *transit, Elm_Transit_Effect_Wipe_Type type, Elm_Transit_Effect_Wipe_Dir dir)
 {
    ELM_TRANSIT_CHECK_OR_RETURN(transit, NULL);
-   void *effect_context = _transit_effect_wipe_context_new(type, dir);
+   void *effect = _transit_effect_wipe_context_new(type, dir);
 
-   if (!effect_context) return NULL;
+   if (!effect)
+     {
+        ERR("Failed to allocate wipe effect! : transit=%p", transit);
+        return NULL;
+     }
    elm_transit_effect_add(transit,
-                          _transit_effect_wipe_op, effect_context,
+                          _transit_effect_wipe_op, effect,
                           _transit_effect_wipe_context_free);
-   return effect_context;
+   return effect;
 }
 
-
-///////////////////////////////////////////////////////////////////////////////
+///////////////////////////////////////////////////////////////////////////
 //Color Effect
-///////////////////////////////////////////////////////////////////////////////
+///////////////////////////////////////////////////////////////////////////
 typedef struct _Elm_Transit_Effect_Color Elm_Transit_Effect_Color;
 
 struct _Elm_Transit_Effect_Color
@@ -1672,18 +1729,22 @@ EAPI Elm_Transit_Effect *
 elm_transit_effect_color_add(Elm_Transit *transit, unsigned int from_r, unsigned int from_g, unsigned int from_b, unsigned int from_a, unsigned int to_r, unsigned int to_g, unsigned int to_b, unsigned int to_a)
 {
    ELM_TRANSIT_CHECK_OR_RETURN(transit, NULL);
-   Elm_Transit_Effect *effect_context = _transit_effect_color_context_new(from_r, from_g, from_b, from_a, to_r, to_g, to_b, to_a);
+   Elm_Transit_Effect *effect = _transit_effect_color_context_new(from_r, from_g, from_b, from_a, to_r, to_g, to_b, to_a);
 
-   if (!effect_context) return NULL;
+   if (!effect)
+     {
+        ERR("Failed to allocate color effect! : transit=%p", transit);
+        return NULL;
+     }
    elm_transit_effect_add(transit,
-                          _transit_effect_color_op, effect_context,
+                          _transit_effect_color_op, effect,
                           _transit_effect_color_context_free);
-   return effect_context;
+   return effect;
 }
 
-///////////////////////////////////////////////////////////////////////////////
+///////////////////////////////////////////////////////////////////////////
 //Fade Effect
-///////////////////////////////////////////////////////////////////////////////
+///////////////////////////////////////////////////////////////////////////
 typedef struct _Elm_Transit_Effect_Fade Elm_Transit_Effect_Fade;
 typedef struct _Elm_Transit_Effect_Fade_Node Elm_Transit_Effect_Fade_Node;
 
@@ -1862,18 +1923,22 @@ elm_transit_effect_fade_add(Elm_Transit *transit)
 {
    ELM_TRANSIT_CHECK_OR_RETURN(transit, NULL);
 
-   Elm_Transit_Effect *effect_context = _transit_effect_fade_context_new();
-   if (!effect_context) return NULL;
+   Elm_Transit_Effect *effect = _transit_effect_fade_context_new();
+
+   if (!effect)
+     {
+        ERR("Failed to allocate fade effect! : transit=%p", transit);
+        return NULL;
+     }
    elm_transit_effect_add(transit,
-                          _transit_effect_fade_op, effect_context,
+                          _transit_effect_fade_op, effect,
                           _transit_effect_fade_context_free);
-   return effect_context;
+   return effect;
 }
 
-
-///////////////////////////////////////////////////////////////////////////////
+///////////////////////////////////////////////////////////////////////////
 //Blend Effect
-///////////////////////////////////////////////////////////////////////////////
+///////////////////////////////////////////////////////////////////////////
 typedef struct _Elm_Transit_Effect_Blend Elm_Transit_Effect_Blend;
 typedef struct _Elm_Transit_Effect_Blend_Node Elm_Transit_Effect_Blend_Node;
 
@@ -2024,19 +2089,22 @@ EAPI Elm_Transit_Effect *
 elm_transit_effect_blend_add(Elm_Transit *transit)
 {
    ELM_TRANSIT_CHECK_OR_RETURN(transit, NULL);
-   Elm_Transit_Effect *effect_context = _transit_effect_blend_context_new();
+   Elm_Transit_Effect *effect = _transit_effect_blend_context_new();
 
-   if (!effect_context) return NULL;
+   if (!effect)
+     {
+        ERR("Failed to allocate blend effect! : transit=%p", transit);
+        return NULL;
+     }
    elm_transit_effect_add(transit,
-                          _transit_effect_blend_op, effect_context,
+                          _transit_effect_blend_op, effect,
                           _transit_effect_blend_context_free);
-   return effect_context;
+   return effect;
 }
 
-
-///////////////////////////////////////////////////////////////////////////////
+///////////////////////////////////////////////////////////////////////////
 //Rotation Effect
-///////////////////////////////////////////////////////////////////////////////
+///////////////////////////////////////////////////////////////////////////
 typedef struct _Elm_Transit_Effect_Rotation Elm_Transit_Effect_Rotation;
 
 struct _Elm_Transit_Effect_Rotation
@@ -2103,19 +2171,22 @@ EAPI Elm_Transit_Effect *
 elm_transit_effect_rotation_add(Elm_Transit *transit, float from_degree, float to_degree)
 {
    ELM_TRANSIT_CHECK_OR_RETURN(transit, NULL);
-   Elm_Transit_Effect *effect_context = _transit_effect_rotation_context_new(from_degree, to_degree);
+   Elm_Transit_Effect *effect = _transit_effect_rotation_context_new(from_degree, to_degree);
 
-   if (!effect_context) return NULL;
+   if (!effect)
+     {
+        ERR("Failed to allocate rotation effect! : transit=%p", transit);
+        return NULL;
+     }
    elm_transit_effect_add(transit,
-                          _transit_effect_rotation_op, effect_context,
+                          _transit_effect_rotation_op, effect,
                           _transit_effect_rotation_context_free);
-   return effect_context;
+   return effect;
 }
 
-
-///////////////////////////////////////////////////////////////////////////////
+///////////////////////////////////////////////////////////////////////////
 //ImageAnimation Effect
-///////////////////////////////////////////////////////////////////////////////
+///////////////////////////////////////////////////////////////////////////
 typedef struct _Elm_Transit_Effect_Image_Animation Elm_Transit_Effect_Image_Animation;
 
 struct _Elm_Transit_Effect_Image_Animation
@@ -2161,7 +2232,7 @@ _transit_effect_image_animation_op(Elm_Transit_Effect *effect, Elm_Transit *tran
 
    EINA_LIST_FOREACH(transit->objs, elist, obj)
      {
-        if (elm_widget_type_check(obj, type))
+        if (elm_widget_type_check(obj, type, __func__))
           elm_icon_file_set(obj,
                             eina_list_nth(image_animation->images, count), NULL);
      }
@@ -2186,7 +2257,11 @@ elm_transit_effect_image_animation_add(Elm_Transit *transit, Eina_List *images)
    ELM_TRANSIT_CHECK_OR_RETURN(transit, NULL);
    Elm_Transit_Effect *effect = _transit_effect_image_animation_context_new(images);
 
-   if (!effect) return NULL;
+   if (!effect)
+     {
+        ERR("Failed to allocate image_animation effect! : transit=%p", transit);
+        return NULL;
+     }
    elm_transit_effect_add(transit,
                           _transit_effect_image_animation_op, effect,
                           _transit_effect_image_animation_context_free);