backend: add default clear animations
[platform/core/uifw/libpui.git] / src / PUI_ani.c
index 9ef7c0f..e42a4cc 100644 (file)
 #include "PUI.h"
 #include <Eina.h>
 
-static int KEY_WL_BUFFER = 0xabcdbeaf;
-static int KEY_CLIENT = 0xdcbabeaf;
+pui_int_error
+_pui_ani_error_to_int_error(pui_error e)
+{
+       pui_int_error ei;
+
+       switch (e)
+       {
+               case PUI_ERROR_NONE:
+                       ei = PUI_INT_ERROR_NONE;
+                       break;
+
+               case PUI_ERROR_INVALID_HANDLE:
+                       ei = PUI_INT_ERROR_INVALID_HANDLE;
+                       break;
+
+               case PUI_ERROR_INVALID_SURFACE:
+                       ei = PUI_INT_ERROR_INVALID_SURFACE;
+                       break;
+
+               case PUI_ERROR_INVALID_BUFFER:
+                       ei = PUI_INT_ERROR_INVALID_BUFFER;
+                       break;
+
+               default:
+                       ei = PUI_INT_ERROR_UNKNOWN;
+                       break;
+       }
+
+       return ei;
+}
 
 static void
 _pui_ani_cb_frame_done(Ecore_Wl2_Window *win, uint32_t timestamp EINA_UNUSED, void *data)
@@ -44,67 +72,38 @@ _pui_ani_cb_frame_done(Ecore_Wl2_Window *win, uint32_t timestamp EINA_UNUSED, vo
        return;
 }
 
-static void
-_buffer_release(void *data, struct wl_buffer *buffer)
-{
-       tbm_surface_h surface = (tbm_surface_h)data;
-       pui_h handle;
-
-       tbm_surface_internal_get_user_data(surface, (unsigned long)&KEY_CLIENT, (void **)&handle);
-       tbm_surface_queue_release(handle->tbm_queue, surface);
-
-       pui_info("[UPDATE] release wl_buffer:%p, surface:%p\n", buffer, surface);
-}
-
-static const struct wl_buffer_listener buffer_listener = {
-    _buffer_release
-};
-
 pui_ani_control_buffer *
-pui_ani_get_buffer(pui_ani_h ani_h)
+pui_ani_get_last_buffer(pui_ani_h ani_h)
 {
        pui_h handle = NULL;
-       tbm_surface_error_e ret;
-       tbm_surface_h surface;
        pui_ani_control_buffer *buffer = NULL;
 
-       if (!ani_h)
+       if (!PUI_MAGIC_CHECK(ani_h, PUI_MAGIC_ANI_H))
        {
-               pui_err("Invalid pui ani handle !\n");
+               PUI_MAGIC_FAIL(ani_h, PUI_MAGIC_ANI_H, __FUNCTION__);
                return NULL;
        }
 
        handle = ani_h->pui_handle;
+       buffer = pui_display_get_last_buffer(handle);
 
-       if (!handle)
-       {
-               pui_err("Error : PUI_INT_ERROR_INVALID_HANDLE\n");
-               return NULL;
-       }
-
-       if (handle->current_surface)
-       {
-               pui_warn("Current_surface is not used !\n");
-       }
-
-       if (!tbm_surface_queue_can_dequeue(handle->tbm_queue, 0))
-       {
-               pui_err("[UPDATE] Cannot dequeue (error : PUI_ERROR_INTERNAL)\n");
-               return NULL;
-       }
+       return buffer;
+}
 
-       ret = tbm_surface_queue_dequeue(handle->tbm_queue, &surface);
+pui_ani_control_buffer *
+pui_ani_get_buffer(pui_ani_h ani_h)
+{
+       pui_h handle = NULL;
+       pui_ani_control_buffer *buffer = NULL;
 
-       if (ret != TBM_SURFACE_ERROR_NONE)
+       if (!PUI_MAGIC_CHECK(ani_h, PUI_MAGIC_ANI_H))
        {
-               pui_err("[UPDATE] dequeue err:%d\n", ret);
+               PUI_MAGIC_FAIL(ani_h, PUI_MAGIC_ANI_H, __FUNCTION__);
                return NULL;
        }
 
-       tbm_surface_map(surface, TBM_SURF_OPTION_READ | TBM_SURF_OPTION_WRITE, &handle->current_sinfo);
-
-       handle->current_surface = surface;
-       buffer = (pui_ani_control_buffer *)&(handle->current_sinfo.planes[0]);
+       handle = ani_h->pui_handle;
+       buffer = pui_display_get_buffer(handle);
 
        return buffer;
 }
@@ -112,43 +111,32 @@ pui_ani_get_buffer(pui_ani_h ani_h)
 pui_int_error
 pui_ani_set_buffer(pui_ani_h ani_h, pui_ani_control_buffer *buffer)
 {
+       pui_error e = PUI_ERROR_NONE;
        pui_h handle = NULL;
 
-       if (!ani_h)
+       if (!PUI_MAGIC_CHECK(ani_h, PUI_MAGIC_ANI_H))
        {
-               pui_err("Invalid ani handle !\n");
-               return PUI_INT_ERROR_INVALID_HANDLE;
+               PUI_MAGIC_FAIL(ani_h, PUI_MAGIC_ANI_H, __FUNCTION__);
+               return PUI_ERROR_INVALID_HANDLE;
        }
 
        handle = ani_h->pui_handle;
 
-       if (!handle)
-       {
-               pui_err("Error : PUI_INT_ERROR_INVALID_HANDLE\n");
-               return PUI_INT_ERROR_INVALID_HANDLE;
-       }
-
-       if (!handle->current_surface)
+       if (!PUI_MAGIC_CHECK(handle, PUI_MAGIC_PUI_H))
        {
-               pui_err("Current_surface is not valid !\n");
-               return PUI_INT_ERROR_INVALID_SURFACE;
+               PUI_MAGIC_FAIL(handle, PUI_MAGIC_PUI_H, __FUNCTION__);
+               return PUI_ERROR_INVALID_HANDLE;
        }
 
-       if (!buffer || !buffer->ptr || !buffer->size)
-               return PUI_INT_ERROR_INVALID_BUFFER;
-
-       handle->is_buffer_set = 1;
+       e = pui_display_set_buffer(handle, buffer);
 
-       return PUI_INT_ERROR_NONE;
+       return _pui_ani_error_to_int_error(e);
 }
 
 pui_int_error
 pui_ani_update(pui_ani_h ani_h)
 {
-       tbm_surface_h surface;
-       tbm_surface_error_e ret;
-       struct wl_buffer *wl_buffer = NULL;
-
+       pui_error e = PUI_ERROR_NONE;
        pui_h handle = NULL;
 
        if (!ani_h)
@@ -158,87 +146,22 @@ pui_ani_update(pui_ani_h ani_h)
        }
 
        handle = ani_h->pui_handle;
+       e = pui_display_update(handle);
 
-       if (!handle)
-       {
-               pui_err("Error : PUI_INT_ERROR_INVALID_HANDLE\n");
-               return PUI_INT_ERROR_INVALID_HANDLE;
-       }
-
-       if (!handle->current_surface)
-               return PUI_INT_ERROR_INVALID_SURFACE;
-
-       if (!handle->is_buffer_set)
-       {
-               pui_err("Buffer is not set !\n");
-               return PUI_INT_ERROR_INVALID_BUFFER;
-       }
-
-       surface = handle->current_surface;
-       handle->current_surface = NULL;
-
-       tbm_surface_unmap(surface);
-
-       ret = tbm_surface_queue_enqueue(handle->tbm_queue, surface);
-
-       if (ret != TBM_SURFACE_ERROR_NONE)
-       {
-               pui_err("[UPDATE] enqueue err:%d\n", ret);
-               return PUI_INT_ERROR_INVALID_SURFACE;
-       }
-
-       ret = tbm_surface_queue_acquire(handle->tbm_queue, &surface);
-
-       if (ret != TBM_SURFACE_ERROR_NONE)
-       {
-               pui_err("[UPDATE] acquire err:%d\n", ret);
-               return PUI_INT_ERROR_INVALID_SURFACE;
-       }
-
-       if (!tbm_surface_internal_get_user_data(surface, (unsigned long)&KEY_WL_BUFFER, (void **)&wl_buffer)) {
-               wl_buffer = wayland_tbm_client_create_buffer(handle->wl_tbm_client, surface);
-
-               if (!wl_buffer)
-               {
-                       pui_err("[UPDATE] failed to create wl_buffer tbm_surface:%p\n", surface);
-                       return PUI_INT_ERROR_INVALID_BUFFER;
-               }
-
-               wl_buffer_add_listener(wl_buffer, &buffer_listener, surface);
-
-               tbm_surface_internal_add_user_data(surface, (unsigned long)&KEY_WL_BUFFER, NULL);
-               tbm_surface_internal_set_user_data(surface, (unsigned long)&KEY_WL_BUFFER, wl_buffer);
-               tbm_surface_internal_add_user_data(surface, (unsigned long)&KEY_CLIENT, NULL);
-               tbm_surface_internal_set_user_data(surface, (unsigned long)&KEY_CLIENT, handle);
-       }
-
-       if (!wl_buffer)
-       {
-               pui_err("[UPDATE] dequeue err:%d\n", ret);
-               return PUI_INT_ERROR_INVALID_BUFFER;
-       }
-
-       ecore_wl2_window_buffer_attach(handle->win, wl_buffer, 0, 0, 0);
-       ecore_wl2_window_damage(handle->win, NULL, 0);
-       ecore_wl2_window_commit(handle->win, EINA_TRUE);
-
-       pui_info("[UPDATE] commit wl_buffer:%p, surface:%p\n", wl_buffer, surface);
-
-       handle->is_buffer_set = 0;
-
-       return PUI_INT_ERROR_NONE;
+       return _pui_ani_error_to_int_error(e);
 }
 
 static Eina_Bool
 _pui_ani_frame_cb(void *data)
 {
        Eina_Bool ret;
+       Ecore_Timer *timer = NULL;
 
        pui_ani_t *ani = (pui_ani_t *)data;
 
-       if (!ani)
+       if (!PUI_MAGIC_CHECK(ani, PUI_MAGIC_ANI_T))
        {
-               pui_err("Invalid pui ani !\n");
+               PUI_MAGIC_FAIL(ani, PUI_MAGIC_ANI_T, __FUNCTION__);
                return ECORE_CALLBACK_CANCEL;
        }
 
@@ -270,7 +193,20 @@ _pui_ani_frame_cb(void *data)
        if (ret && PUI_ANI_STATUS_STARTED == ani->status)
                pui_ani_status_update(ani, PUI_ANI_STATUS_RUNNING);
 
-       return EINA_TRUE;
+       ecore_timer_del(ani->frame_cb_timer);
+       ani->frame_cb_timer = NULL;
+
+       timer = ecore_timer_add(ani->frame_interval, _pui_ani_frame_cb, ani);
+
+       if (!timer)
+       {
+               pui_err("Failed to add ecore timer !\n");
+               return 0;
+       }
+
+       ani->frame_cb_timer = timer;
+
+       return EINA_FALSE;
 }
 
 pui_bool
@@ -278,9 +214,9 @@ pui_ani_add_frame_cb(pui_ani_t *ani, pui_bool (*frame_cb)(void *data, int serial
 {
        Ecore_Timer *timer = NULL;
 
-       if (!ani)
+       if (!PUI_MAGIC_CHECK(ani, PUI_MAGIC_ANI_T))
        {
-               pui_err("Invalid put ani !\n");
+               PUI_MAGIC_FAIL(ani, PUI_MAGIC_ANI_T, __FUNCTION__);
                return 0;
        }
 
@@ -317,9 +253,9 @@ pui_ani_add_frame_cb(pui_ani_t *ani, pui_bool (*frame_cb)(void *data, int serial
 void
 pui_ani_remove_frame_cb(pui_ani_t *ani)
 {
-       if (!ani)
+       if (!PUI_MAGIC_CHECK(ani, PUI_MAGIC_ANI_T))
        {
-               pui_err("Invalid put ani !\n");
+               PUI_MAGIC_FAIL(ani, PUI_MAGIC_ANI_T, __FUNCTION__);
                return;
        }
 
@@ -340,8 +276,17 @@ pui_ani_remove_frame_cb(pui_ani_t *ani)
 pui_id
 pui_ani_get_id(pui_ani_h ani_h)
 {
-       if (!ani_h || !ani_h->ani)
+       if (!PUI_MAGIC_CHECK(ani_h, PUI_MAGIC_ANI_H))
+       {
+               PUI_MAGIC_FAIL(ani_h, PUI_MAGIC_ANI_H, __FUNCTION__);
                return NULL;
+       }
+
+       if (!PUI_MAGIC_CHECK(ani_h->ani, PUI_MAGIC_ANI_T))
+       {
+               PUI_MAGIC_FAIL(ani_h->ani, PUI_MAGIC_ANI_T, __FUNCTION__);
+               return NULL;
+       }
 
        return ani_h->ani->id;
 }
@@ -349,8 +294,17 @@ pui_ani_get_id(pui_ani_h ani_h)
 pui_ani_cmd
 pui_ani_get_cmd(pui_ani_h ani_h)
 {
-       if (!ani_h || !ani_h->ani)
+       if (!PUI_MAGIC_CHECK(ani_h, PUI_MAGIC_ANI_H))
+       {
+               PUI_MAGIC_FAIL(ani_h, PUI_MAGIC_ANI_H, __FUNCTION__);
                return PUI_ANI_CMD_NONE;
+       }
+
+       if (!PUI_MAGIC_CHECK(ani_h->ani, PUI_MAGIC_ANI_T))
+       {
+               PUI_MAGIC_FAIL(ani_h->ani, PUI_MAGIC_ANI_T, __FUNCTION__);
+               return PUI_ANI_CMD_NONE;
+       }
 
        return ani_h->ani->cmd;
 }
@@ -358,8 +312,17 @@ pui_ani_get_cmd(pui_ani_h ani_h)
 int
 pui_ani_get_repeat(pui_ani_h ani_h)
 {
-       if (!ani_h || !ani_h->ani)
+       if (!PUI_MAGIC_CHECK(ani_h, PUI_MAGIC_ANI_H))
+       {
+               PUI_MAGIC_FAIL(ani_h, PUI_MAGIC_ANI_H, __FUNCTION__);
                return 0;
+       }
+
+       if (!PUI_MAGIC_CHECK(ani_h->ani, PUI_MAGIC_ANI_T))
+       {
+               PUI_MAGIC_FAIL(ani_h->ani, PUI_MAGIC_ANI_T, __FUNCTION__);
+               return 0;
+       }
 
        return ani_h->ani->repeat;
 }
@@ -367,8 +330,11 @@ pui_ani_get_repeat(pui_ani_h ani_h)
 pui_backend_ani_data *
 pui_ani_get_ani_data(pui_ani_t *ani)
 {
-       if (!ani)
+       if (!PUI_MAGIC_CHECK(ani, PUI_MAGIC_ANI_T))
+       {
+               PUI_MAGIC_FAIL(ani, PUI_MAGIC_ANI_T, __FUNCTION__);
                return NULL;
+       }
 
        return ani->ani_data;
 }
@@ -380,9 +346,9 @@ pui_ani_status_update(pui_ani_t *ani, pui_ani_status status)
        pui_ani_h ani_h;
        PUI_Event_Animation_Status *e = NULL;
 
-       if (!ani)
+       if (!PUI_MAGIC_CHECK(ani, PUI_MAGIC_ANI_T))
        {
-               pui_err("Invalid pui ani !\n");
+               PUI_MAGIC_FAIL(ani, PUI_MAGIC_ANI_T, __FUNCTION__);
                return;
        }
 
@@ -391,18 +357,7 @@ pui_ani_status_update(pui_ani_t *ani, pui_ani_status status)
 
        ani_h = ani->ani_h;
 
-       e = (PUI_Event_Animation_Status *)calloc(1, sizeof(PUI_Event_Animation_Status));
-
-       if (!e)
-       {
-               pui_err("Failed to allocate memory for PUI Event !\n");
-               return;
-       }
-
        ani->status = status;
-       e->ani_h = ani_h;
-       e->win = ecore_wl2_window_id_get(ani_h->pui_handle->win);
-       e->status = status;
 
        switch (status)
        {
@@ -432,6 +387,18 @@ pui_ani_status_update(pui_ani_t *ani, pui_ani_status status)
 
        if (ev_type > 0)
        {
+               e = (PUI_Event_Animation_Status *)calloc(1, sizeof(PUI_Event_Animation_Status));
+
+               if (!e)
+               {
+                       pui_err("Failed to allocate memory for PUI Event !\n");
+                       return;
+               }
+
+               e->ani_h = ani_h;
+               e->win = ecore_wl2_window_id_get(ani_h->pui_handle->win);
+               e->status = status;
+
                ecore_event_add(ev_type, e, NULL, ani_h);
        }
 }
@@ -441,9 +408,9 @@ pui_ani_status_get(pui_ani_t *ani)
 {
        pui_ani_status status = PUI_ANI_STATUS_UNKNOWN;
 
-       if (!ani)
+       if (!PUI_MAGIC_CHECK(ani, PUI_MAGIC_ANI_T))
        {
-               pui_err("Invalid pui ani !\n");
+               PUI_MAGIC_FAIL(ani, PUI_MAGIC_ANI_T, __FUNCTION__);
                return status;
        }
 
@@ -458,8 +425,11 @@ _pui_ani_control_with_force(pui_ani_h ani_h, pui_ani_cmd cmd, int repeat, pui_bo
        pui_h handle = NULL;
        pui_backend_ani_func *ani_func = NULL;
 
-       if (!ani_h)
+       if (!PUI_MAGIC_CHECK(ani_h, PUI_MAGIC_ANI_H))
+       {
+               PUI_MAGIC_FAIL(ani_h, PUI_MAGIC_ANI_H, __FUNCTION__);
                return PUI_ERROR_INVALID_ANI_HANDLE;
+       }
 
        if (cmd < PUI_ANI_CMD_START || cmd >= PUI_ANI_CMD_LAST)
        {
@@ -476,9 +446,21 @@ _pui_ani_control_with_force(pui_ani_h ani_h, pui_ani_cmd cmd, int repeat, pui_bo
        handle = ani_h->pui_handle;
        ani = ani_h->ani;
 
-       if (!ani || !ani->ani_data)
+       if (!PUI_MAGIC_CHECK(handle, PUI_MAGIC_PUI_H))
+       {
+               PUI_MAGIC_FAIL(handle, PUI_MAGIC_PUI_H, __FUNCTION__);
+               return PUI_ERROR_INTERNAL;
+       }
+
+       if (!PUI_MAGIC_CHECK(ani, PUI_MAGIC_ANI_T))
        {
-               pui_err("Invalid ani or ani_data !\n");
+               PUI_MAGIC_FAIL(ani, PUI_MAGIC_ANI_T, __FUNCTION__);
+               return PUI_ERROR_INTERNAL;
+       }
+
+       if (!ani->ani_data)
+       {
+               pui_err("Invalid ani_data !\n");
                return PUI_ERROR_INTERNAL;
        }
 
@@ -489,6 +471,12 @@ _pui_ani_control_with_force(pui_ani_h ani_h, pui_ani_cmd cmd, int repeat, pui_bo
 
        if (cmd == PUI_ANI_CMD_START)
        {
+               if (handle->manual_render)
+               {
+                       pui_err("Manual render has been set ! Unable to start animation.\n");
+                       return PUI_ERROR_MANUAL_RENDER_ENABLED;
+               }
+
                if (handle->current_ani_h && handle->current_ani_h != ani_h)
                {
                        pui_ani_t *current_ani = handle->current_ani_h->ani;
@@ -526,6 +514,9 @@ _pui_ani_control_with_force(pui_ani_h ani_h, pui_ani_cmd cmd, int repeat, pui_bo
        {
                ei = ani_func->ani_stop(ani, force);
 
+               if (ani->frame_cb_timer)
+                       pui_ani_remove_frame_cb(ani);
+
                if (ei != PUI_INT_ERROR_NONE)
                {
                        pui_err("Failied on stopping animation ! (id:%s)\n", ani->id);
@@ -533,9 +524,6 @@ _pui_ani_control_with_force(pui_ani_h ani_h, pui_ani_cmd cmd, int repeat, pui_bo
                        if (ani->status != PUI_ANI_STATUS_STOPPED)
                                pui_ani_status_update(ani, PUI_ANI_STATUS_STOPPED);
 
-                       if (ani->frame_cb_timer)
-                               pui_ani_remove_frame_cb(ani);
-
                        return PUI_ERROR_INTERNAL;
                }
 
@@ -572,6 +560,24 @@ _cb_visibility_change(void *data, int type EINA_UNUSED, void *event)
        Ecore_Wl2_Event_Window_Visibility_Change *ev;
        PUI_Event_Animation_Status *e = NULL;
 
+       if (!PUI_MAGIC_CHECK(ani_h, PUI_MAGIC_ANI_H))
+       {
+               PUI_MAGIC_FAIL(ani_h, PUI_MAGIC_ANI_H, __FUNCTION__);
+               return ECORE_CALLBACK_PASS_ON;
+       }
+
+       if (!PUI_MAGIC_CHECK(ani, PUI_MAGIC_ANI_T))
+       {
+               PUI_MAGIC_FAIL(ani, PUI_MAGIC_ANI_T, __FUNCTION__);
+               return ECORE_CALLBACK_PASS_ON;
+       }
+
+       if (!PUI_MAGIC_CHECK(ph, PUI_MAGIC_PUI_H))
+       {
+               PUI_MAGIC_FAIL(ph, PUI_MAGIC_PUI_H, __FUNCTION__);
+               return ECORE_CALLBACK_PASS_ON;
+       }
+
        ev = event;
 
        /* check if this is needed */
@@ -618,9 +624,9 @@ _pui_ani_event_handlers_init(pui_ani_h ani_h)
 {
        Ecore_Event_Handler *h = NULL;
 
-       if (!ani_h)
+       if (!PUI_MAGIC_CHECK(ani_h, PUI_MAGIC_ANI_H))
        {
-               pui_err("Invalid handle !\n");
+               PUI_MAGIC_FAIL(ani_h, PUI_MAGIC_ANI_H, __FUNCTION__);
                return;
        }
 
@@ -635,9 +641,9 @@ _pui_ani_event_handlers_init(pui_ani_h ani_h)
 static void
 _pui_ani_event_handlers_shutdown(pui_ani_h ani_h)
 {
-       if (!ani_h)
+       if (!PUI_MAGIC_CHECK(ani_h, PUI_MAGIC_ANI_H))
        {
-               pui_err("Invalid handle !\n");
+               PUI_MAGIC_FAIL(ani_h, PUI_MAGIC_ANI_H, __FUNCTION__);
                return;
        }
 
@@ -658,9 +664,15 @@ pui_ani_create(pui_h handle, pui_id id)
        pui_ani_t *ani = NULL;
        pui_backend_ani_data *ani_data = NULL;
 
-       if (!handle || !handle->backend_module_data)
+       if (!PUI_MAGIC_CHECK(handle, PUI_MAGIC_PUI_H))
        {
-               pui_err("Invalid pui handle or backend module data !\n");
+               PUI_MAGIC_FAIL(handle, PUI_MAGIC_PUI_H, __FUNCTION__);
+               return NULL;
+       }
+
+       if (!handle->backend_module_data)
+       {
+               pui_err("Invalid backend module data !\n");
                return NULL;
        }
 
@@ -684,6 +696,8 @@ pui_ani_create(pui_h handle, pui_id id)
        ani_h->pui_handle = handle;
        ani_h->ecore_event_hdls = NULL;
 
+       PUI_MAGIC_SET(ani_h, PUI_MAGIC_ANI_H);
+
        _pui_ani_event_handlers_init(ani_h);
 
        ani = (pui_ani_t *)calloc(1, sizeof(pui_ani_t));
@@ -705,6 +719,8 @@ pui_ani_create(pui_h handle, pui_id id)
 
        handle->ani_handles = eina_list_append(handle->ani_handles, ani_h);
 
+       PUI_MAGIC_SET(ani, PUI_MAGIC_ANI_T);
+
        return ani_h;
 
 err:
@@ -714,7 +730,10 @@ err:
        }
 
        if (ani_h)
+       {
+               PUI_MAGIC_SET(ani_h, PUI_MAGIC_NONE);
                free(ani_h);
+       }
 
        return NULL;
 }
@@ -726,12 +745,27 @@ pui_ani_destroy(pui_ani_h ani_h)
        pui_ani_t *ani = NULL;
        pui_backend_module_data *backend_module_data = NULL;
 
-       if (!ani_h || !ani_h->pui_handle)
+       if (!PUI_MAGIC_CHECK(ani_h, PUI_MAGIC_ANI_H))
+       {
+               PUI_MAGIC_FAIL(ani_h, PUI_MAGIC_ANI_H, __FUNCTION__);
                return;
+       }
 
        handle = ani_h->pui_handle;
        ani = ani_h->ani;
 
+       if (!PUI_MAGIC_CHECK(handle, PUI_MAGIC_PUI_H))
+       {
+               PUI_MAGIC_FAIL(handle, PUI_MAGIC_PUI_H, __FUNCTION__);
+               return;
+       }
+
+       if (!PUI_MAGIC_CHECK(ani, PUI_MAGIC_ANI_T))
+       {
+               PUI_MAGIC_FAIL(ani, PUI_MAGIC_ANI_T, __FUNCTION__);
+               return;
+       }
+
        /* stop the animation being played already if any */
        if (ani->status == PUI_ANI_STATUS_STARTED || ani->status == PUI_ANI_STATUS_RUNNING)
                pui_ani_control(ani_h, PUI_ANI_CMD_STOP, 0);
@@ -746,6 +780,7 @@ pui_ani_destroy(pui_ani_h ani_h)
                ani->frame_cb_timer = NULL;
        }
 
+       PUI_MAGIC_SET(ani_h->ani, PUI_MAGIC_NONE);
        free(ani_h->ani);
 
        _pui_ani_event_handlers_shutdown(ani_h);
@@ -758,6 +793,8 @@ pui_ani_destroy(pui_ani_h ani_h)
 
        handle->ani_handles = eina_list_remove(handle->ani_handles, ani_h);
 
+       PUI_MAGIC_SET(ani_h, PUI_MAGIC_NONE);
+
        free(ani_h->id);
        free(ani_h);
 }