Change ROI mode behavior for player ROI precondition

Change-Id: Iaacf183d43ecdf226029f55d19dabfb9de83a918
Signed-off-by: Hyunil <hyunil46.park@samsung.com>

diff --git a/packaging/libmm-player.spec b/packaging/libmm-player.spec
index d2dceba..b00325a 100644 (file)
--- a/packaging/libmm-player.spec
+++ b/packaging/libmm-player.spec
@@ -1,6 +1,6 @@
 Name:       libmm-player
 Summary:    Multimedia Framework Player Library
-Version:    0.6.99
+Version:    0.6.100
 Release:    0
 Group:      Multimedia/Libraries
 License:    Apache-2.0

diff --git a/src/mm_player_priv.c b/src/mm_player_priv.c
index 5658983..b9d303a 100755 (executable)
--- a/src/mm_player_priv.c
+++ b/src/mm_player_priv.c
@@ -3892,7 +3892,7 @@ __mmplayer_video_param_set_display_method(mm_player_t* player)
        LOGD("set video param : method %d", display_method);
 }
 
-int
+void
 __mmplayer_video_param_set_render_rectangle(mm_player_t* player)
 {
        MMHandleType attrs = 0;
@@ -3906,10 +3906,10 @@ __mmplayer_video_param_set_render_rectangle(mm_player_t* player)
 
        /* check video sinkbin is created */
        if (MM_ERROR_NONE != __mmplayer_video_param_check_video_sink_bin(player))
-               return MM_ERROR_PLAYER_INTERNAL;
+               return;
 
        attrs = MMPLAYER_GET_ATTRS(player);
-       MMPLAYER_RETURN_VAL_IF_FAIL(attrs, MM_ERROR_PLAYER_INTERNAL);
+       MMPLAYER_RETURN_IF_FAIL(attrs);
 
        mm_attrs_get_data_by_name(attrs, "display_overlay", &handle);
 
@@ -3926,10 +3926,8 @@ __mmplayer_video_param_set_render_rectangle(mm_player_t* player)
                         wl_window_x, wl_window_y, wl_window_width, wl_window_height);
                LOGD("set video param : render rectangle : x(%d) y(%d) width(%d) height(%d)",
                        wl_window_x, wl_window_y, wl_window_width, wl_window_height);
-       } else {
-               return MM_ERROR_PLAYER_INTERNAL;
+
        }
-       return MM_ERROR_NONE;
 }
 void
 __mmplayer_video_param_set_display_overlay(mm_player_t* player)
@@ -3965,8 +3963,6 @@ int
 __mmplayer_update_wayland_videosink_video_param(mm_player_t* player, char *param_name)
 {
        bool update_all_param = FALSE;
-       MMHandleType attrs = 0;
-       int display_method = 0;
        MMPLAYER_FENTER();
 
        /* check video sinkbin is created */
@@ -3986,14 +3982,8 @@ __mmplayer_update_wayland_videosink_video_param(mm_player_t* player, char *param
                __mmplayer_video_param_set_display_overlay(player);
        if (update_all_param || !g_strcmp0(param_name, "display_method"))
                __mmplayer_video_param_set_display_method(player);
-       /* check roi mode is set */
-       attrs = MMPLAYER_GET_ATTRS(player);
-       MMPLAYER_RETURN_VAL_IF_FAIL(attrs, MM_ERROR_PLAYER_INTERNAL);
-       mm_attrs_get_int_by_name(attrs, "display_method", &display_method);
-       if ((display_method == PLAYER_DISPLAY_MODE_DST_ROI) && (update_all_param || !g_strcmp0(param_name, "wl_window_render_x"))) {
-               if (MM_ERROR_NONE != __mmplayer_video_param_set_render_rectangle(player))
-                       return MM_ERROR_PLAYER_INTERNAL;
-       }
+       if (update_all_param || !g_strcmp0(param_name, "wl_window_render_x"))
+               __mmplayer_video_param_set_render_rectangle(player);
        if (update_all_param || !g_strcmp0(param_name, "display_visible"))
                __mmplayer_video_param_set_display_visible(player);
        if (update_all_param || !g_strcmp0(param_name, "display_rotation"))