Interface to provider is changed
[platform/framework/web/livebox-viewer.git] / src / livebox.c
index c3a112a..8af633d 100644 (file)
@@ -58,7 +58,7 @@ static struct info {
        .fault_list = NULL,
        .init_count = 0,
        .prevent_overwrite = 0,
-       .event_filter = 0.02f,
+       .event_filter = 0.01f,
 };
 
 struct cb_info {
@@ -160,7 +160,7 @@ static void update_mode_cb(struct livebox *handler, const struct packet *result,
        }
 
        if (ret < 0) {
-               DbgPrint("Resize request is failed: %d\n", ret);
+               ErrPrint("Resize request is failed: %d\n", ret);
                goto errout;
        }
 
@@ -176,13 +176,6 @@ errout:
 static void resize_cb(struct livebox *handler, const struct packet *result, void *data)
 {
        int ret;
-       struct cb_info *info = data;
-       ret_cb_t cb;
-       void *cbdata;
-
-       cb = info->cb;
-       cbdata = info->data;
-       destroy_cb_info(info);
 
        if (!result) {
                ret = LB_STATUS_ERROR_FAULT;
@@ -203,11 +196,10 @@ static void resize_cb(struct livebox *handler, const struct packet *result, void
         * after this request.
         */
        if (ret < 0) {
-               DbgPrint("Resize request is failed: %d\n", ret);
+               ErrPrint("Resize request is failed: %d\n", ret);
                goto errout;
        }
 
-       DbgPrint("Resize request is successfully sent\n");
        return;
 
 errout:
@@ -234,8 +226,9 @@ static void text_signal_cb(struct livebox *handler, const struct packet *result,
                ret = LB_STATUS_ERROR_INVALID;
        }
 
-       if (cb)
+       if (cb) {
                cb(handler, ret, cbdata);
+       }
        return;
 }
 
@@ -252,8 +245,9 @@ static void set_group_ret_cb(struct livebox *handler, const struct packet *resul
                goto errout;
        }
 
-       if (ret < 0)
+       if (ret < 0) {
                goto errout;
+       }
 
        return;
 
@@ -276,10 +270,10 @@ static void period_ret_cb(struct livebox *handler, const struct packet *result,
                goto errout;
        }
 
-       if (ret < 0)
+       if (ret < 0) {
                goto errout;
+       }
 
-       DbgPrint("Succeed to send period change request, waiting result\n");
        return;
 
 errout:
@@ -308,7 +302,6 @@ static void del_ret_cb(struct livebox *handler, const struct packet *result, voi
        }
 
        if (ret == 0) {
-               DbgPrint("Returns %d (waiting deleted event)\n", ret);
                handler->deleted_cb = cb;
                handler->deleted_cbdata = cbdata;
        } else if (cb) {
@@ -344,7 +337,6 @@ static void new_ret_cb(struct livebox *handler, const struct packet *result, voi
        }
 
        if (ret >= 0) {
-               DbgPrint("new request is sent, just waiting the created event\n");
                handler->created_cb = cb;
                handler->created_cbdata = cbdata;
 
@@ -379,7 +371,7 @@ static void pd_create_cb(struct livebox *handler, const struct packet *result, v
        }
 
        if (ret < 0) {
-               DbgPrint("Failed to create a PD\n");
+               ErrPrint("Failed to create a PD[%d]\n", ret);
                goto errout;
        }
 
@@ -409,8 +401,9 @@ static void activated_cb(struct livebox *handler, const struct packet *result, v
                ret = LB_STATUS_ERROR_INVALID;
        }
 
-       if (cb)
+       if (cb) {
                cb(handler, ret, cbdata);
+       }
 }
 
 static void pd_destroy_cb(struct livebox *handler, const struct packet *result, void *data)
@@ -425,19 +418,17 @@ static void pd_destroy_cb(struct livebox *handler, const struct packet *result,
        destroy_cb_info(info);
 
        if (!result) {
-               DbgPrint("Result is NIL (may connection lost)\n");
+               ErrPrint("Result is NIL (may connection lost)\n");
                ret = LB_STATUS_ERROR_FAULT;
        } else if (packet_get(result, "i", &ret) != 1) {
-               DbgPrint("Invalid parameter\n");
+               ErrPrint("Invalid parameter\n");
                ret = LB_STATUS_ERROR_INVALID;
        }
 
        if (ret == 0) {
-               DbgPrint("PD Destroyed callback prepared\n");
                handler->pd_destroyed_cb = cb;
                handler->pd_destroyed_cbdata = cbdata;
        } else if (cb) {
-               DbgPrint("PD is not desroyed (forcely reset, pd flag)\n");
                handler->is_pd_created = 0;
                cb(handler, ret, cbdata);
        }
@@ -460,10 +451,9 @@ static void delete_cluster_cb(struct livebox *handler, const struct packet *resu
                ret = LB_STATUS_ERROR_INVALID;
        }
 
-       DbgPrint("Delete category returns: %d\n", ret);
-
-       if (cb)
+       if (cb) {
                cb(handler, ret, cbdata);
+       }
 }
 
 static void delete_category_cb(struct livebox *handler, const struct packet *result, void *data)
@@ -477,19 +467,20 @@ static void delete_category_cb(struct livebox *handler, const struct packet *res
        cbdata = info->data;
        destroy_cb_info(info);
 
-       if (!result)
+       if (!result) {
                ret = LB_STATUS_ERROR_FAULT;
-       else if (packet_get(result, "i", &ret) != 1)
+       } else if (packet_get(result, "i", &ret) != 1) {
                ret = LB_STATUS_ERROR_INVALID;
+       }
 
-       DbgPrint("Delete category returns: %d\n", ret);
-
-       if (cb)
+       if (cb) {
                cb(handler, ret, cbdata);
+       }
 }
 
-static void pixmap_acquired_cb(struct livebox *handler, const struct packet *result, void *data)
+static void lb_pixmap_acquired_cb(struct livebox *handler, const struct packet *result, void *data)
 {
+       int pixmap;
        int ret;
        ret_cb_t cb;
        void *cbdata;
@@ -499,13 +490,53 @@ static void pixmap_acquired_cb(struct livebox *handler, const struct packet *res
        cbdata = info->data;
        destroy_cb_info(info);
 
-       if (!result)
-               ret = 0; /* PIXMAP 0 means error */
-       else if (packet_get(result, "i", &ret) != 1)
-               ret = 0;
+       if (!result) {
+               pixmap = 0; /* PIXMAP 0 means error */
+       } else if (packet_get(result, "ii", &pixmap, &ret) != 2) {
+               pixmap = 0;
+       }
 
-       if (cb)
-               cb(handler, ret, cbdata);
+       if (ret == LB_STATUS_ERROR_BUSY) {
+               ret = livebox_acquire_lb_pixmap(handler, cb, cbdata);
+               DbgPrint("Busy, Try again: %d\n", ret);
+               /* Try again */
+       } else {
+               if (cb) {
+                       cb(handler, pixmap, cbdata);
+               }
+       }
+}
+
+static void pd_pixmap_acquired_cb(struct livebox *handler, const struct packet *result, void *data)
+{
+       int pixmap;
+       int ret;
+       ret_cb_t cb;
+       void *cbdata;
+       struct cb_info *info = data;
+
+       cb = info->cb;
+       cbdata = info->data;
+       destroy_cb_info(info);
+
+       if (!result) {
+               pixmap = 0; /* PIXMAP 0 means error */
+               ret = LB_STATUS_ERROR_FAULT;
+       } else if (packet_get(result, "ii", &pixmap, &ret) != 2) {
+               pixmap = 0;
+               ret = LB_STATUS_ERROR_INVALID;
+       }
+
+       if (ret == LB_STATUS_ERROR_BUSY) {
+               ret = livebox_acquire_pd_pixmap(handler, cb, cbdata);
+               DbgPrint("Busy, Try again: %d\n", ret);
+               /* Try again */
+       } else {
+               if (cb) {
+                       DbgPrint("ret: %d, pixmap: %d\n", ret, pixmap);
+                       cb(handler, pixmap, cbdata);
+               }
+       }
 }
 
 static void pinup_done_cb(struct livebox *handler, const struct packet *result, void *data)
@@ -519,8 +550,9 @@ static void pinup_done_cb(struct livebox *handler, const struct packet *result,
                goto errout;
        }
 
-       if (ret < 0)
+       if (ret < 0) {
                goto errout;
+       }
 
        return;
 
@@ -548,7 +580,6 @@ static void access_ret_cb(struct livebox *handler, const struct packet *result,
                goto errout;
        }
 
-       DbgPrint("Access event is successfully sent. waiting result\n");
        return;
 
 errout:
@@ -571,8 +602,6 @@ static int send_access_event(struct livebox *handler, const char *event, int x,
                return LB_STATUS_ERROR_FAULT;
        }
 
-       DbgPrint("Send: %dx%d\n", x, y);
-
        return master_rpc_async_request(handler, packet, 0, access_ret_cb, NULL);
 }
 
@@ -588,8 +617,6 @@ static int send_mouse_event(struct livebox *handler, const char *event, int x, i
                return LB_STATUS_ERROR_FAULT;
        }
 
-       DbgPrint("Send: %dx%d\n", x, y);
-
        return master_rpc_request_only(handler, packet);
 }
 
@@ -602,19 +629,22 @@ EAPI int livebox_init(void *disp)
                return LB_STATUS_SUCCESS;
        }
        env = getenv("PROVIDER_DISABLE_PREVENT_OVERWRITE");
-       if (env && !strcasecmp(env, "true"))
+       if (env && !strcasecmp(env, "true")) {
                s_info.prevent_overwrite = 1;
+       }
 
        env = getenv("PROVIDER_EVENT_FILTER");
-       if (env)
+       if (env) {
                sscanf(env, "%lf", &MINIMUM_EVENT);
+       }
 
 #if defined(FLOG)
        char filename[BUFSIZ];
        snprintf(filename, sizeof(filename), "/tmp/%d.box.log", getpid());
        __file_log_fp = fopen(filename, "w+t");
-       if (!__file_log_fp)
+       if (!__file_log_fp) {
                __file_log_fp = fdopen(1, "w+t");
+       }
 #endif
        critical_log_init("viewer");
        livebox_service_init();
@@ -629,13 +659,13 @@ EAPI int livebox_init(void *disp)
 EAPI int livebox_fini(void)
 {
        if (s_info.init_count <= 0) {
-               DbgPrint("Didn't initialized\n");
+               ErrPrint("Doesn't initialized\n");
                return LB_STATUS_ERROR_INVALID;
        }
 
        s_info.init_count--;
        if (s_info.init_count > 0) {
-               DbgPrint("init count : %d\n", s_info.init_count);
+               ErrPrint("init count : %d\n", s_info.init_count);
                return LB_STATUS_SUCCESS;
        }
 
@@ -652,8 +682,9 @@ static inline char *lb_pkgname(const char *pkgname)
 
        lb = livebox_service_pkgname(pkgname);
        if (!lb) {
-               if (util_validate_livebox_package(pkgname) == 0)
+               if (util_validate_livebox_package(pkgname) == 0) {
                        return strdup(pkgname);
+               }
        }
 
        return lb;
@@ -682,8 +713,9 @@ EAPI struct livebox *livebox_add_with_size(const char *pkgname, const char *cont
                return NULL;
        }
 
-       if (type != LB_SIZE_TYPE_UNKNOWN)
+       if (type != LB_SIZE_TYPE_UNKNOWN) {
                livebox_service_get_size(type, &width, &height);
+       }
 
        handler = calloc(1, sizeof(*handler));
        if (!handler) {
@@ -735,8 +767,9 @@ EAPI struct livebox *livebox_add_with_size(const char *pkgname, const char *cont
                return NULL;
        }
 
-       if (!cb)
+       if (!cb) {
                cb = default_create_cb;
+       }
 
        /* Data provider will set this */
        handler->lb.type = _LB_TYPE_FILE;
@@ -790,7 +823,6 @@ EAPI struct livebox *livebox_add_with_size(const char *pkgname, const char *cont
                return NULL;
        }
 
-       DbgPrint("Successfully sent a new request ([%lf] %s)\n", handler->timestamp, handler->pkgname);
        handler->state = CREATE;
        return lb_ref(handler);
 }
@@ -836,16 +868,14 @@ EAPI int livebox_set_period(struct livebox *handler, double period, ret_cb_t cb,
                return LB_STATUS_ERROR_FAULT;
        }
 
-       if (!cb)
+       if (!cb) {
                cb = default_period_changed_cb;
-
-       handler->period_changed_cb = cb;
-       handler->period_cbdata = data;
+       }
 
        ret = master_rpc_async_request(handler, packet, 0, period_ret_cb, NULL);
-       if (ret < 0) {
-               handler->period_changed_cb = NULL;
-               handler->period_cbdata = NULL;
+       if (ret == LB_STATUS_SUCCESS) {
+               handler->period_changed_cb = cb;
+               handler->period_cbdata = data;
        }
 
        return ret;
@@ -875,13 +905,15 @@ EAPI int livebox_del(struct livebox *handler, ret_cb_t cb, void *data)
                 * By the way, if the user adds any callback for getting return status of this,
                 * call it at here.
                 */
-               if (cb)
+               if (cb) {
                        cb(handler, 0, data);
+               }
                return LB_STATUS_SUCCESS;
        }
 
-       if (!cb)
+       if (!cb) {
                cb = default_delete_cb;
+       }
 
        return lb_send_delete(handler, cb, data);
 }
@@ -890,8 +922,9 @@ EAPI int livebox_set_fault_handler(int (*cb)(enum livebox_fault_type, const char
 {
        struct fault_info *info;
 
-       if (!cb)
+       if (!cb) {
                return LB_STATUS_ERROR_INVALID;
+       }
 
        info = malloc(sizeof(*info));
        if (!info) {
@@ -914,6 +947,7 @@ EAPI void *livebox_unset_fault_handler(int (*cb)(enum livebox_fault_type, const
        dlist_foreach(s_info.fault_list, l, info) {
                if (info->handler == cb) {
                        void *data;
+
                        s_info.fault_list = dlist_remove(s_info.fault_list, l);
                        data = info->user_data;
                        free(info);
@@ -977,34 +1011,36 @@ EAPI int livebox_set_update_mode(struct livebox *handler, int active_update, ret
                return LB_STATUS_ERROR_INVALID;
        }
 
-       if (handler->state != CREATE || !handler->id)
+       if (handler->state != CREATE || !handler->id) {
                return LB_STATUS_ERROR_INVALID;
+       }
 
        if (handler->update_mode_cb) {
                ErrPrint("Previous update_mode cb is not finished yet\n");
                return LB_STATUS_ERROR_BUSY;
        }
 
-       if (handler->is_active_update == active_update)
+       if (handler->is_active_update == active_update) {
                return LB_STATUS_ERROR_ALREADY;
+       }
 
-       if (!handler->is_user)
+       if (!handler->is_user) {
                return LB_STATUS_ERROR_PERMISSION;
+       }
 
        packet = packet_create("update_mode", "ssi", handler->pkgname, handler->id, active_update);
-       if (!packet)
+       if (!packet) {
                return LB_STATUS_ERROR_FAULT;
+       }
 
-       if (!cb)
+       if (!cb) {
                cb = default_update_mode_cb;
-
-       handler->update_mode_cb = cb;
-       handler->update_mode_cbdata = data;
+       }
 
        ret = master_rpc_async_request(handler, packet, 0, update_mode_cb, NULL);
-       if (ret < 0) {
-               handler->update_mode_cb = NULL;
-               handler->update_mode_cbdata = NULL;
+       if (ret == LB_STATUS_SUCCESS) {
+               handler->update_mode_cb = cb;
+               handler->update_mode_cbdata = data;
        }
 
        return ret;
@@ -1017,8 +1053,9 @@ EAPI int livebox_is_active_update(struct livebox *handler)
                return LB_STATUS_ERROR_INVALID;
        }
 
-       if (handler->state != CREATE || !handler->id)
+       if (handler->state != CREATE || !handler->id) {
                return LB_STATUS_ERROR_INVALID;
+       }
 
        return handler->is_active_update;
 }
@@ -1066,16 +1103,14 @@ EAPI int livebox_resize(struct livebox *handler, int type, ret_cb_t cb, void *da
                return LB_STATUS_ERROR_FAULT;
        }
 
-       if (!cb)
+       if (!cb) {
                cb = default_lb_size_changed_cb;
-
-       handler->size_changed_cb = cb;
-       handler->size_cbdata = data;
+       }
 
        ret = master_rpc_async_request(handler, packet, 0, resize_cb, NULL);
-       if (ret < 0) {
-               handler->size_changed_cb = NULL;
-               handler->size_cbdata = NULL;
+       if (ret == LB_STATUS_SUCCESS) {
+               handler->size_changed_cb = cb;
+               handler->size_cbdata = data;
        }
 
        return ret;
@@ -1088,7 +1123,6 @@ EAPI int livebox_click(struct livebox *handler, double x, double y)
        int ret;
 
        timestamp = util_timestamp();
-       DbgPrint("CLICKED: %lf\n", timestamp);
 
        if (!handler) {
                ErrPrint("Handler is NIL\n");
@@ -1101,9 +1135,10 @@ EAPI int livebox_click(struct livebox *handler, double x, double y)
        }
 
        if (handler->lb.auto_launch) {
-               DbgPrint("Auto-launch enabled: %s\n", handler->lb.auto_launch);
-               if (aul_launch_app(handler->lb.auto_launch, NULL) < 0)
+               DbgPrint("AUTO_LAUNCH [%s]\n", handler->lb.auto_launch);
+               if (aul_launch_app(handler->lb.auto_launch, NULL) < 0) {
                        ErrPrint("Failed to launch app %s\n", handler->lb.auto_launch);
+               }
        }
 
        packet = packet_create_noack("clicked", "sssddd", handler->pkgname, handler->id, "clicked", timestamp, x, y);
@@ -1112,22 +1147,25 @@ EAPI int livebox_click(struct livebox *handler, double x, double y)
                return LB_STATUS_ERROR_FAULT;
        }
 
+       DbgPrint("CLICKED: %lf\n", timestamp);
        ret = master_rpc_request_only(handler, packet);
-       DbgPrint("Click request: %d\n", ret);
 
        if (!handler->lb.mouse_event && (handler->lb.type == _LB_TYPE_BUFFER || handler->lb.type == _LB_TYPE_SCRIPT)) {
                int ret; /* Shadow variable */
                ret = send_mouse_event(handler, "lb_mouse_down", x * handler->lb.width, y * handler->lb.height);
-               if (ret < 0)
-                       DbgPrint("Failed to send Down: %d\n", ret);
+               if (ret < 0) {
+                       ErrPrint("Failed to send Down: %d\n", ret);
+               }
 
                ret = send_mouse_event(handler, "lb_mouse_move", x * handler->lb.width, y * handler->lb.height);
-               if (ret < 0)
-                       DbgPrint("Failed to send Move: %d\n", ret);
+               if (ret < 0) {
+                       ErrPrint("Failed to send Move: %d\n", ret);
+               }
 
                ret = send_mouse_event(handler, "lb_mouse_up", x * handler->lb.width, y * handler->lb.height);
-               if (ret < 0)
-                       DbgPrint("Failed to send Up: %d\n", ret);
+               if (ret < 0) {
+                       ErrPrint("Failed to send Up: %d\n", ret);
+               }
        }
 
        return ret;
@@ -1199,16 +1237,15 @@ EAPI int livebox_create_pd_with_position(struct livebox *handler, double x, doub
                return LB_STATUS_ERROR_FAULT;
        }
 
-       if (!cb)
+       if (!cb) {
                cb = default_pd_created_cb;
+       }
 
-       handler->pd_created_cb = cb;
-       handler->pd_created_cbdata = data;
-
+       DbgPrint("PERF_DBOX\n");
        ret = master_rpc_async_request(handler, packet, 0, pd_create_cb, NULL);
-       if (ret < 0) {
-               handler->pd_created_cb = NULL;
-               handler->pd_created_cbdata = NULL;
+       if (ret == LB_STATUS_SUCCESS) {
+               handler->pd_created_cb = cb;
+               handler->pd_created_cbdata = data;
        }
 
        return ret;
@@ -1229,7 +1266,7 @@ EAPI int livebox_move_pd(struct livebox *handler, double x, double y)
        }
 
        if (!handler->is_pd_created) {
-               DbgPrint("PD is not created\n");
+               ErrPrint("PD is not created\n");
                return LB_STATUS_ERROR_INVALID;
        }
 
@@ -1248,8 +1285,9 @@ EAPI int livebox_activate(const char *pkgname, ret_cb_t cb, void *data)
        struct cb_info *cbinfo;
        int ret;
 
-       if (!pkgname)
+       if (!pkgname) {
                return LB_STATUS_ERROR_INVALID;
+       }
 
        packet = packet_create("activate_package", "s", pkgname);
        if (!packet) {
@@ -1265,8 +1303,9 @@ EAPI int livebox_activate(const char *pkgname, ret_cb_t cb, void *data)
        }
 
        ret = master_rpc_async_request(NULL, packet, 0, activated_cb, cbinfo);
-       if (ret < 0)
+       if (ret < 0) {
                destroy_cb_info(cbinfo);
+       }
 
        return ret;
 }
@@ -1298,8 +1337,9 @@ EAPI int livebox_destroy_pd(struct livebox *handler, ret_cb_t cb, void *data)
                return LB_STATUS_ERROR_FAULT;
        }
 
-       if (!cb)
+       if (!cb) {
                cb = default_pd_destroyed_cb;
+       }
 
        cbinfo = create_cb_info(cb, data);
        if (!cbinfo) {
@@ -1308,8 +1348,9 @@ EAPI int livebox_destroy_pd(struct livebox *handler, ret_cb_t cb, void *data)
        }
 
        ret = master_rpc_async_request(handler, packet, 0, pd_destroy_cb, cbinfo);
-       if (ret < 0)
+       if (ret < 0) {
                destroy_cb_info(cbinfo);
+       }
 
        return ret;
 }
@@ -1391,16 +1432,14 @@ EAPI int livebox_access_event(struct livebox *handler, enum access_event_type ty
                return LB_STATUS_ERROR_INVALID;
        }
 
-       if (!cb)
+       if (!cb) {
                cb = default_access_event_cb;
-
-       handler->access_event_cb = cb;
-       handler->access_event_cbdata = data;
+       }
 
        ret = send_access_event(handler, cmd, x * w, y * h);
-       if (ret < 0) {
-               handler->access_event_cb = NULL;
-               handler->access_event_cbdata = NULL;
+       if (ret == LB_STATUS_SUCCESS) {
+               handler->access_event_cb = cb;
+               handler->access_event_cbdata = data;
        }
 
        return ret;
@@ -1438,8 +1477,9 @@ EAPI int livebox_content_event(struct livebox *handler, enum content_event_type
                        }
 
                        if (type & CONTENT_EVENT_MOUSE_MOVE) {
-                               if (fabs(x - handler->pd.x) < MINIMUM_EVENT && fabs(y - handler->pd.y) < MINIMUM_EVENT)
+                               if (fabs(x - handler->pd.x) < MINIMUM_EVENT && fabs(y - handler->pd.y) < MINIMUM_EVENT) {
                                        return LB_STATUS_ERROR_BUSY;
+                               }
                        } else if (type & CONTENT_EVENT_MOUSE_SET) {
                                flag = 0;
                        }
@@ -1458,7 +1498,6 @@ EAPI int livebox_content_event(struct livebox *handler, enum content_event_type
 
                if (type & CONTENT_EVENT_MOUSE_MASK) {
                        if (!handler->lb.mouse_event) {
-                               ErrPrint("Box is not support the mouse event\n");
                                return LB_STATUS_ERROR_INVALID;
                        }
 
@@ -1468,8 +1507,9 @@ EAPI int livebox_content_event(struct livebox *handler, enum content_event_type
                        }
 
                        if (type & CONTENT_EVENT_MOUSE_MOVE) {
-                               if (fabs(x - handler->lb.x) < MINIMUM_EVENT && fabs(y - handler->lb.y) < MINIMUM_EVENT)
+                               if (fabs(x - handler->lb.x) < MINIMUM_EVENT && fabs(y - handler->lb.y) < MINIMUM_EVENT) {
                                        return LB_STATUS_ERROR_BUSY;
+                               }
                        } else if (type & CONTENT_EVENT_MOUSE_SET) {
                                flag = 0;
                        }
@@ -1539,8 +1579,9 @@ EAPI const char *livebox_filename(struct livebox *handler)
                return NULL;
        }
 
-       if (handler->filename)
+       if (handler->filename) {
                return handler->filename;
+       }
 
        /* Oooops */
        return util_uri_to_path(handler->id);
@@ -1561,13 +1602,14 @@ EAPI int livebox_get_pdsize(struct livebox *handler, int *w, int *h)
                return LB_STATUS_ERROR_INVALID;
        }
 
-       if (!w)
+       if (!w) {
                w = &_w;
-       if (!h)
+       }
+       if (!h) {
                h = &_h;
+       }
 
        if (!handler->is_pd_created) {
-               DbgPrint("Buffer is not created yet [%dx%d]\n", *w, *h);
                *w = handler->pd.default_width;
                *h = handler->pd.default_height;
        } else {
@@ -1600,7 +1642,6 @@ EAPI int livebox_size(struct livebox *handler)
        case _LB_TYPE_BUFFER:
        case _LB_TYPE_SCRIPT:
                if (!fb_is_created(handler->lb.data.fb)) {
-                       DbgPrint("Buffer is not created yet - reset size\n");
                        w = 0;
                        h = 0;
                }
@@ -1648,16 +1689,14 @@ EAPI int livebox_set_group(struct livebox *handler, const char *cluster, const c
                return LB_STATUS_ERROR_FAULT;
        }
 
-       if (!cb)
+       if (!cb) {
                cb = default_group_changed_cb;
-
-       handler->group_changed_cb = cb;
-       handler->group_cbdata = data; 
+       }
 
        ret = master_rpc_async_request(handler, packet, 0, set_group_ret_cb, NULL);
-       if (ret < 0) {
-               handler->group_changed_cb = NULL;
-               handler->group_cbdata = NULL;
+       if (ret == LB_STATUS_SUCCESS) {
+               handler->group_changed_cb = cb;
+               handler->group_cbdata = data; 
        }
 
        return ret;
@@ -1697,8 +1736,9 @@ EAPI int livebox_get_supported_sizes(struct livebox *handler, int *cnt, int *siz
 
        for (j = i = 0; i < NR_OF_SIZE_LIST; i++) {
                if (handler->lb.size_list & (0x01 << i)) {
-                       if (j == *cnt)
+                       if (j == *cnt) {
                                break;
+                       }
 
                        size_list[j++] = (0x01 << i);
                }
@@ -1757,8 +1797,9 @@ EAPI int livebox_delete_cluster(const char *cluster, ret_cb_t cb, void *data)
        }
 
        ret = master_rpc_async_request(NULL, packet, 0, delete_cluster_cb, cbinfo);
-       if (ret < 0)
+       if (ret < 0) {
                destroy_cb_info(cbinfo);
+       }
 
        return ret;
 }
@@ -1782,8 +1823,9 @@ EAPI int livebox_delete_category(const char *cluster, const char *category, ret_
        }
 
        ret = master_rpc_async_request(NULL, packet, 0, delete_category_cb, cbinfo);
-       if (ret < 0)
+       if (ret < 0) {
                destroy_cb_info(cbinfo);
+       }
 
        return ret;
 }
@@ -1808,8 +1850,9 @@ EAPI enum livebox_lb_type livebox_lb_type(struct livebox *handler)
                {
                        const char *id;
                        id = fb_id(handler->lb.data.fb);
-                       if (id && !strncasecmp(id, SCHEMA_PIXMAP, strlen(SCHEMA_PIXMAP)))
+                       if (id && !strncasecmp(id, SCHEMA_PIXMAP, strlen(SCHEMA_PIXMAP))) {
                                return LB_TYPE_PIXMAP;
+                       }
                }
                return LB_TYPE_BUFFER;
        case _LB_TYPE_TEXT:
@@ -1841,8 +1884,9 @@ EAPI enum livebox_pd_type livebox_pd_type(struct livebox *handler)
                {
                        const char *id;
                        id = fb_id(handler->pd.data.fb);
-                       if (id && !strncasecmp(id, SCHEMA_PIXMAP, strlen(SCHEMA_PIXMAP)))
+                       if (id && !strncasecmp(id, SCHEMA_PIXMAP, strlen(SCHEMA_PIXMAP))) {
                                return PD_TYPE_PIXMAP;
+                       }
                }
                return PD_TYPE_BUFFER;
        default:
@@ -1907,8 +1951,9 @@ EAPI int livebox_acquire_lb_pixmap(struct livebox *handler, ret_cb_t cb, void *d
        }
 
        id = fb_id(handler->lb.data.fb);
-       if (!id || strncasecmp(id, SCHEMA_PIXMAP, strlen(SCHEMA_PIXMAP)))
+       if (!id || strncasecmp(id, SCHEMA_PIXMAP, strlen(SCHEMA_PIXMAP))) {
                return LB_STATUS_ERROR_INVALID;
+       }
 
        packet = packet_create("lb_acquire_pixmap", "ss", handler->pkgname, handler->id);
        if (!packet) {
@@ -1922,9 +1967,10 @@ EAPI int livebox_acquire_lb_pixmap(struct livebox *handler, ret_cb_t cb, void *d
                return LB_STATUS_ERROR_FAULT;
        }
 
-       ret = master_rpc_async_request(handler, packet, 0, pixmap_acquired_cb, cbinfo);
-       if (ret < 0)
+       ret = master_rpc_async_request(handler, packet, 0, lb_pixmap_acquired_cb, cbinfo);
+       if (ret < 0) {
                destroy_cb_info(cbinfo);
+       }
 
        return ret;
 }
@@ -1933,8 +1979,8 @@ EAPI int livebox_release_lb_pixmap(struct livebox *handler, int pixmap)
 {
        struct packet *packet;
 
-       if (!handler) {
-               ErrPrint("Handler is NIL\n");
+       if (!handler || pixmap == 0) {
+               ErrPrint("Handler is NIL [%d]\n", pixmap);
                return LB_STATUS_ERROR_INVALID;
        }
 
@@ -1980,8 +2026,9 @@ EAPI int livebox_acquire_pd_pixmap(struct livebox *handler, ret_cb_t cb, void *d
        }
 
        id = fb_id(handler->pd.data.fb);
-       if (!id || strncasecmp(id, SCHEMA_PIXMAP, strlen(SCHEMA_PIXMAP)))
+       if (!id || strncasecmp(id, SCHEMA_PIXMAP, strlen(SCHEMA_PIXMAP))) {
                return LB_STATUS_ERROR_INVALID;
+       }
 
        packet = packet_create("pd_acquire_pixmap", "ss", handler->pkgname, handler->id);
        if (!packet) {
@@ -1995,9 +2042,10 @@ EAPI int livebox_acquire_pd_pixmap(struct livebox *handler, ret_cb_t cb, void *d
                return LB_STATUS_ERROR_FAULT;
        }
 
-       ret = master_rpc_async_request(handler, packet, 0, pixmap_acquired_cb, cbinfo);
-       if (ret < 0)
+       ret = master_rpc_async_request(handler, packet, 0, pd_pixmap_acquired_cb, cbinfo);
+       if (ret < 0) {
                destroy_cb_info(cbinfo);
+       }
 
        return ret;
 }
@@ -2064,8 +2112,8 @@ EAPI int livebox_release_pd_pixmap(struct livebox *handler, int pixmap)
 {
        struct packet *packet;
 
-       if (!handler) {
-               ErrPrint("Handler is NIL\n");
+       if (!handler || pixmap == 0) {
+               ErrPrint("Handler is NIL [%d]\n", pixmap);
                return LB_STATUS_ERROR_INVALID;
        }
 
@@ -2224,16 +2272,14 @@ EAPI int livebox_set_pinup(struct livebox *handler, int flag, ret_cb_t cb, void
                return LB_STATUS_ERROR_FAULT;
        }
 
-       if (!cb)
+       if (!cb) {
                cb = default_pinup_cb;
-
-       handler->pinup_cb = cb;
-       handler->pinup_cbdata = data;
+       }
 
        ret = master_rpc_async_request(handler, packet, 0, pinup_done_cb, NULL);
-       if (ret < 0) {
-               handler->pinup_cb = NULL;
-               handler->pinup_cbdata = NULL;
+       if (ret == LB_STATUS_SUCCESS) {
+               handler->pinup_cb = cb;
+               handler->pinup_cbdata = data;
        }
 
        return ret;
@@ -2246,8 +2292,9 @@ EAPI int livebox_is_pinned_up(struct livebox *handler)
                return LB_STATUS_ERROR_INVALID;
        }
 
-       if (handler->state != CREATE || !handler->id)
+       if (handler->state != CREATE || !handler->id) {
                return LB_STATUS_ERROR_INVALID;
+       }
 
        return handler->is_pinned_up;
 }
@@ -2259,8 +2306,9 @@ EAPI int livebox_has_pinup(struct livebox *handler)
                return LB_STATUS_ERROR_INVALID;
        }
 
-       if (handler->state != CREATE || !handler->id)
+       if (handler->state != CREATE || !handler->id) {
                return LB_STATUS_ERROR_INVALID;
+       }
 
        return handler->lb.pinup_supported;
 }
@@ -2272,8 +2320,9 @@ EAPI int livebox_set_data(struct livebox *handler, void *data)
                return LB_STATUS_ERROR_INVALID;
        }
 
-       if (handler->state != CREATE)
+       if (handler->state != CREATE) {
                return LB_STATUS_ERROR_INVALID;
+       }
 
        handler->data = data;
        return LB_STATUS_SUCCESS;
@@ -2286,8 +2335,9 @@ EAPI void *livebox_get_data(struct livebox *handler)
                return NULL;
        }
 
-       if (handler->state != CREATE)
+       if (handler->state != CREATE) {
                return NULL;
+       }
 
        return handler->data;
 }
@@ -2312,8 +2362,9 @@ EAPI const char *livebox_content(struct livebox *handler)
                return NULL;
        }
 
-       if (handler->state != CREATE)
+       if (handler->state != CREATE) {
                return NULL;
+       }
 
        return handler->content;
 }
@@ -2325,8 +2376,9 @@ EAPI const char *livebox_category_title(struct livebox *handler)
                return NULL;
        }
 
-       if (handler->state != CREATE)
+       if (handler->state != CREATE) {
                return NULL;
+       }
 
        return handler->title;
 }
@@ -2347,11 +2399,13 @@ EAPI int livebox_emit_text_signal(struct livebox *handler, const char *emission,
                return LB_STATUS_ERROR_INVALID;
        }
 
-       if (!emission)
+       if (!emission) {
                emission = "";
+       }
 
-       if (!source)
+       if (!source) {
                source = "";
+       }
 
        packet = packet_create("text_signal", "ssssdddd",
                                handler->pkgname, handler->id, emission, source, sx, sy, ex, ey);
@@ -2367,8 +2421,9 @@ EAPI int livebox_emit_text_signal(struct livebox *handler, const char *emission,
        }
 
        ret = master_rpc_async_request(handler, packet, 0, text_signal_cb, cbinfo);
-       if (ret < 0)
+       if (ret < 0) {
                destroy_cb_info(cbinfo);
+       }
 
        return LB_STATUS_ERROR_FAULT;
 }
@@ -2421,8 +2476,9 @@ EAPI int livebox_refresh(struct livebox *handler)
                return LB_STATUS_ERROR_INVALID;
        }
 
-       if (handler->state != CREATE || !handler->id)
+       if (handler->state != CREATE || !handler->id) {
                return LB_STATUS_ERROR_INVALID;
+       }
 
        packet = packet_create_noack("update", "ss", handler->pkgname, handler->id);
        if (!packet) {
@@ -2461,8 +2517,9 @@ EAPI int livebox_set_visibility(struct livebox *handler, enum livebox_visible_st
                return LB_STATUS_ERROR_INVALID;
        }
 
-       if (handler->state != CREATE || !handler->id)
+       if (handler->state != CREATE || !handler->id) {
                return LB_STATUS_ERROR_INVALID;
+       }
 
        if (!handler->is_user) {
                /* System cluster livebox cannot be changed its visible states */
@@ -2472,10 +2529,9 @@ EAPI int livebox_set_visibility(struct livebox *handler, enum livebox_visible_st
                }
        }
 
-       DbgPrint("Change the visibility %d <> %d, %s\n", handler->visible, state, handler->id);
-
-       if (handler->visible == state)
+       if (handler->visible == state) {
                return LB_STATUS_ERROR_ALREADY;
+       }
 
        packet = packet_create_noack("change,visibility", "ssi", handler->pkgname, handler->id, (int)state);
        if (!packet) {
@@ -2484,8 +2540,9 @@ EAPI int livebox_set_visibility(struct livebox *handler, enum livebox_visible_st
        }
 
        ret = master_rpc_request_only(handler, packet);
-       if (ret == 0)
+       if (ret == 0) {
                handler->visible = state;
+       }
 
        return ret;
 }
@@ -2497,8 +2554,9 @@ EAPI enum livebox_visible_state livebox_visibility(struct livebox *handler)
                return LB_VISIBLE_ERROR;
        }
 
-       if (handler->state != CREATE || !handler->id)
+       if (handler->state != CREATE || !handler->id) {
                return LB_VISIBLE_ERROR;
+       }
 
        return handler->visible;
 }
@@ -2525,11 +2583,13 @@ int lb_set_group(struct livebox *handler, const char *cluster, const char *categ
                }
        }
 
-       if (handler->cluster)
+       if (handler->cluster) {
                free(handler->cluster);
+       }
 
-       if (handler->category)
+       if (handler->category) {
                free(handler->category);
+       }
 
        handler->cluster = pc;
        handler->category = ps;
@@ -2567,8 +2627,9 @@ void lb_invoke_fault_handler(enum livebox_fault_type event, const char *pkgname,
        struct fault_info *info;
 
        dlist_foreach_safe(s_info.fault_list, l, n, info) {
-               if (info->handler(event, pkgname, file, func, info->user_data) == EXIT_FAILURE)
+               if (info->handler(event, pkgname, file, func, info->user_data) == EXIT_FAILURE) {
                        s_info.fault_list = dlist_remove(s_info.fault_list, l);
+               }
        }
 }
 
@@ -2579,8 +2640,9 @@ void lb_invoke_event_handler(struct livebox *handler, enum livebox_event_type ev
        struct event_info *info;
 
        dlist_foreach_safe(s_info.event_list, l, n, info) {
-               if (info->handler(handler, event, info->user_data) == EXIT_FAILURE)
+               if (info->handler(handler, event, info->user_data) == EXIT_FAILURE) {
                        s_info.event_list = dlist_remove(s_info.event_list, l);
+               }
        }
 }
 
@@ -2590,11 +2652,13 @@ struct livebox *lb_find_livebox(const char *pkgname, const char *id)
        struct livebox *handler;
 
        dlist_foreach(s_info.livebox_list, l, handler) {
-               if (!handler->id)
+               if (!handler->id) {
                        continue;
+               }
 
-               if (!strcmp(handler->pkgname, pkgname) && !strcmp(handler->id, id))
+               if (!strcmp(handler->pkgname, pkgname) && !strcmp(handler->id, id)) {
                        return handler;
+               }
        }
 
        return NULL;
@@ -2606,8 +2670,9 @@ struct livebox *lb_find_livebox_by_timestamp(double timestamp)
        struct livebox *handler;
 
        dlist_foreach(s_info.livebox_list, l, handler) {
-               if (handler->timestamp == timestamp)
+               if (handler->timestamp == timestamp) {
                        return handler;
+               }
        }
 
        return NULL;
@@ -2631,8 +2696,9 @@ static inline char *get_file_kept_in_safe(const char *id)
         */
        if (s_info.prevent_overwrite) {
                new_path = strdup(path);
-               if (!new_path)
+               if (!new_path) {
                        ErrPrint("Heap: %s\n", strerror(errno));
+               }
 
                return new_path;
        }
@@ -2683,8 +2749,9 @@ struct livebox *lb_new_livebox(const char *pkgname, const char *id, double times
        handler->filename = get_file_kept_in_safe(id);
        if (!handler->filename) {
                handler->filename = strdup(util_uri_to_path(id));
-               if (!handler->filename)
+               if (!handler->filename) {
                        ErrPrint("Error: %s\n", strerror(errno));
+               }
        }
 
        handler->timestamp = timestamp;
@@ -2715,13 +2782,11 @@ int lb_delete_all(void)
 int lb_set_content(struct livebox *handler, const char *content)
 {
        if (handler->content) {
-               DbgPrint("Release content: %s\n", content);
                free(handler->content);
                handler->content = NULL;
        }
 
        if (content) {
-               DbgPrint("Update Content: [%s]\n", content);
                handler->content = strdup(content);
                if (!handler->content) {
                        CRITICAL_LOG("Heap: %s (content: %s)\n", strerror(errno), content);
@@ -2757,12 +2822,14 @@ void lb_set_size_list(struct livebox *handler, int size_list)
 
 void lb_set_auto_launch(struct livebox *handler, const char *auto_launch)
 {
-       if (!strlen(auto_launch))
+       if (!strlen(auto_launch)) {
                return;
+       }
 
        handler->lb.auto_launch = strdup(auto_launch);
-       if (!handler->lb.auto_launch)
+       if (!handler->lb.auto_launch) {
                ErrPrint("Heap: %s\n", strerror(errno));
+       }
 }
 
 void lb_set_priority(struct livebox *handler, double priority)
@@ -2772,21 +2839,25 @@ void lb_set_priority(struct livebox *handler, double priority)
 
 void lb_set_id(struct livebox *handler, const char *id)
 {
-       if (handler->id)
+       if (handler->id) {
                free(handler->id);
+       }
 
        handler->id = strdup(id);
-       if (!handler->id)
+       if (!handler->id) {
                ErrPrint("Error: %s\n", strerror(errno));
+       }
 
-       if (handler->filename)
+       if (handler->filename) {
                free(handler->filename);
+       }
 
        handler->filename = get_file_kept_in_safe(id);
        if (!handler->filename) {
                handler->filename = strdup(util_uri_to_path(id));
-               if (!handler->filename)
+               if (!handler->filename) {
                        ErrPrint("Error: %s\n", strerror(errno));
+               }
        }
 }
 
@@ -2794,31 +2865,36 @@ int lb_set_lb_fb(struct livebox *handler, const char *filename)
 {
        struct fb_info *fb;
 
-       if (!handler)
+       if (!handler) {
                return LB_STATUS_ERROR_INVALID;
+       }
 
        fb = handler->lb.data.fb;
-       if (fb && !strcmp(fb_id(fb), filename)) /*!< BUFFER is not changed, */
+       if (fb && !strcmp(fb_id(fb), filename)) /*!< BUFFER is not changed, */
                return LB_STATUS_SUCCESS;
+       }
 
        handler->lb.data.fb = NULL;
 
        if (!filename || filename[0] == '\0') {
-               if (fb)
+               if (fb) {
                        fb_destroy(fb);
+               }
                return LB_STATUS_SUCCESS;
        }
 
        handler->lb.data.fb = fb_create(filename, handler->lb.width, handler->lb.height);
        if (!handler->lb.data.fb) {
                ErrPrint("Faield to create a FB\n");
-               if (fb)
+               if (fb) {
                        fb_destroy(fb);
+               }
                return LB_STATUS_ERROR_FAULT;
        }
 
-       if (fb)
+       if (fb) {
                fb_destroy(fb);
+       }
 
        return LB_STATUS_SUCCESS;
 }
@@ -2827,8 +2903,9 @@ int lb_set_pd_fb(struct livebox *handler, const char *filename)
 {
        struct fb_info *fb;
 
-       if (!handler)
+       if (!handler) {
                return LB_STATUS_ERROR_INVALID;
+       }
 
        fb = handler->pd.data.fb;
        if (fb && !strcmp(fb_id(fb), filename)) {
@@ -2838,21 +2915,24 @@ int lb_set_pd_fb(struct livebox *handler, const char *filename)
        handler->pd.data.fb = NULL;
 
        if (!filename || filename[0] == '\0') {
-               if (fb)
+               if (fb) {
                        fb_destroy(fb);
+               }
                return LB_STATUS_SUCCESS;
        }
 
        handler->pd.data.fb = fb_create(filename, handler->pd.width, handler->pd.height);
        if (!handler->pd.data.fb) {
                ErrPrint("Failed to create a FB\n");
-               if (fb)
+               if (fb) {
                        fb_destroy(fb);
+               }
                return LB_STATUS_ERROR_FAULT;
        }
 
-       if (fb)
+       if (fb) {
                fb_destroy(fb);
+       }
        return LB_STATUS_SUCCESS;
 }
 
@@ -2903,8 +2983,9 @@ void lb_set_period(struct livebox *handler, double period)
 
 struct livebox *lb_ref(struct livebox *handler)
 {
-       if (!handler)
+       if (!handler) {
                return NULL;
+       }
 
        handler->refcnt++;
        return handler;
@@ -2912,85 +2993,78 @@ struct livebox *lb_ref(struct livebox *handler)
 
 struct livebox *lb_unref(struct livebox *handler)
 {
-       if (!handler)
+       if (!handler) {
                return NULL;
+       }
 
        handler->refcnt--;
-       if (handler->refcnt > 0)
+       if (handler->refcnt > 0) {
                return handler;
+       }
 
        if (handler->created_cb) {
-               DbgPrint("Clear created event callback\n");
                handler->created_cb(handler, LB_STATUS_ERROR_FAULT, handler->created_cbdata);
                handler->created_cb = NULL;
                handler->created_cbdata = NULL;
        }
 
        if (handler->deleted_cb) {
-               DbgPrint("Clear deleted event callback\n");
                handler->deleted_cb(handler, LB_STATUS_ERROR_FAULT, handler->deleted_cbdata);
                handler->deleted_cb = NULL;
                handler->deleted_cbdata = NULL;
        }
 
        if (handler->pinup_cb) {
-               DbgPrint("Clear pinup event callback\n");
                handler->pinup_cb(handler, LB_STATUS_ERROR_FAULT, handler->pinup_cbdata);
                handler->pinup_cb = NULL;
                handler->pinup_cbdata = NULL;
        }
 
        if (handler->group_changed_cb) {
-               DbgPrint("Clear group changed event callback\n");
                handler->group_changed_cb(handler, LB_STATUS_ERROR_FAULT, handler->group_cbdata);
                handler->group_changed_cb = NULL;
                handler->group_cbdata = NULL;
        }
 
        if (handler->period_changed_cb) {
-               DbgPrint("Clear period changed event callback\n");
                handler->period_changed_cb(handler, LB_STATUS_ERROR_FAULT, handler->period_cbdata);
                handler->period_changed_cb = NULL;
                handler->period_cbdata = NULL;
        }
 
        if (handler->size_changed_cb) {
-               DbgPrint("Clear size changed event callback\n");
                handler->size_changed_cb(handler, LB_STATUS_ERROR_FAULT, handler->size_cbdata);
                handler->size_changed_cb = NULL;
                handler->size_cbdata = NULL;
        }
 
        if (handler->pd_created_cb) {
-               DbgPrint("Clear pd created event callback\n");
                handler->pd_created_cb(handler, LB_STATUS_ERROR_FAULT, handler->pd_created_cbdata);
                handler->pd_created_cb = NULL;
                handler->pd_created_cbdata = NULL;
        }
 
        if (handler->pd_destroyed_cb) {
-               DbgPrint("Clear pd destroyed event callback\n");
                handler->pd_destroyed_cb(handler, LB_STATUS_ERROR_FAULT, handler->pd_destroyed_cbdata);
                handler->pd_destroyed_cb = NULL;
                handler->pd_destroyed_cbdata = NULL;
        }
 
        if (handler->update_mode_cb) {
-               DbgPrint("Clear update mode callback\n");
                handler->update_mode_cb(handler, LB_STATUS_ERROR_FAULT, handler->update_mode_cbdata);
                handler->update_mode_cb = NULL;
                handler->update_mode_cbdata = NULL;
        }
 
        if (handler->access_event_cb) {
-               DbgPrint("Clear access status callback\n");
-               handler->access_event_cb(handler, LB_STATUS_ERROR_FAULT, handler->access_event_cbdata);
+               handler->access_event_cb(handler, LB_ACCESS_STATUS_ERROR, handler->access_event_cbdata);
                handler->access_event_cb = NULL;
                handler->access_event_cbdata = NULL;
        }
 
-       if (handler->filename)
+       if (handler->filename) {
                util_unlink(handler->filename);
+       }
 
        dlist_remove_data(s_info.livebox_list, handler);
 
@@ -3035,14 +3109,16 @@ int lb_send_delete(struct livebox *handler, ret_cb_t cb, void *data)
        packet = packet_create("delete", "ss", handler->pkgname, handler->id);
        if (!packet) {
                ErrPrint("Failed to build a param\n");
-               if (cb)
+               if (cb) {
                        cb(handler, LB_STATUS_ERROR_FAULT, data);
+               }
 
                return LB_STATUS_ERROR_FAULT;
        }
 
-       if (!cb)
+       if (!cb) {
                cb = default_delete_cb;
+       }
 
        cbinfo = create_cb_info(cb, data);
        if (!cbinfo) {
@@ -3051,8 +3127,9 @@ int lb_send_delete(struct livebox *handler, ret_cb_t cb, void *data)
        }
 
        ret = master_rpc_async_request(handler, packet, 0, del_ret_cb, cbinfo);
-       if (ret < 0)
+       if (ret < 0) {
                destroy_cb_info(cbinfo);
+       }
 
        return ret;
 }