summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorEunhae Choi <eunhae1.choi@samsung.com>2017-01-13 16:41:26 +0900
committerGilbok Lee <gilbok.lee@samsung.com>2017-01-13 16:48:25 +0900
commit87cc8a9d4592997342bf0f82e217dc34ffae4645 (patch)
tree9555bf7b36ebd9ac4cb745b81fdbc90f683d1bd3
parentb0deccd9d9692201f20d43a29fc5c4d3e5a3768b (diff)
downloadlibmm-player-87cc8a9d4592997342bf0f82e217dc34ffae4645.tar.gz
libmm-player-87cc8a9d4592997342bf0f82e217dc34ffae4645.tar.bz2
libmm-player-87cc8a9d4592997342bf0f82e217dc34ffae4645.zip
[v0.6.22] post resource conflict msg regardless of videobin
Change-Id: I04abd5a1ae2f039292fa8e06356b4d21237a651b
-rw-r--r--src/mm_player_priv.c2
-rw-r--r--src/mm_player_resource.c10
2 files changed, 8 insertions, 4 deletions
diff --git a/src/mm_player_priv.c b/src/mm_player_priv.c
index 74edc32..e15bc45 100644
--- a/src/mm_player_priv.c
+++ b/src/mm_player_priv.c
@@ -1445,7 +1445,7 @@ __mmplayer_gst_callback(GstBus *bus, GstMessage *msg, gpointer data) // @
{
if (MMPLAYER_IS_STREAMING(player)) {
// managed prepare async case when buffering is completed
- // pending state should be reset oyherwise, it's still playing even though it's resumed after bufferging.
+ // pending state should be reset otherwise, it's still playing even though it's resumed after bufferging.
if ((MMPLAYER_CURRENT_STATE(player) != MM_PLAYER_STATE_PLAYING) ||
(MMPLAYER_PENDING_STATE(player) == MM_PLAYER_STATE_PLAYING))
MMPLAYER_SET_STATE(player, MM_PLAYER_STATE_PLAYING);
diff --git a/src/mm_player_resource.c b/src/mm_player_resource.c
index c886b0c..ab95c86 100644
--- a/src/mm_player_resource.c
+++ b/src/mm_player_resource.c
@@ -189,6 +189,7 @@ static void mrp_resource_release_cb(mrp_res_context_t *cx, const mrp_res_resourc
int result = MM_ERROR_NONE;
mm_player_t* player = NULL;
mrp_res_resource_t *res;
+ gboolean resource_released = FALSE;
MMPLAYER_FENTER();
@@ -206,10 +207,13 @@ static void mrp_resource_release_cb(mrp_res_context_t *cx, const mrp_res_resourc
LOGD(" - resource set state of player(%p) is changed to [%s]\n", player, state_to_str(rs->state));
for (i = 0; i < MRP_RESOURCE_MAX; i++) {
res = mrp_res_get_resource_by_name(rs, resource_str[i]);
- if (res == NULL)
+ if (res == NULL) {
LOGW(" -- %s not present in resource set\n", resource_str[i]);
- else
+ } else {
LOGD(" -- resource name [%s] -> [%s]'\n", res->name, state_to_str(res->state));
+ if (res->state == MRP_RES_RESOURCE_ABOUT_TO_LOOSE)
+ resource_released = TRUE;
+ }
}
/* do something to release resource here.
@@ -217,7 +221,7 @@ static void mrp_resource_release_cb(mrp_res_context_t *cx, const mrp_res_resourc
if (!__mmplayer_can_do_interrupt(player)) {
LOGW("no need to interrupt, so leave");
} else {
- if (player->pipeline->videobin) {
+ if (resource_released) {
player->resource_manager.by_rm_cb = TRUE;
LOGD("video resource conflict so, resource will be freed by unrealizing");
result = _mmplayer_unrealize((MMHandleType)player);