[M120 Migration] Reset is_seeking_ in MediaPlayerBridgeCapi::Release() 90/317890/1
authorzhishun.zhou <zhishun.zhou@samsung.com>
Fri, 20 Sep 2024 05:57:38 +0000 (13:57 +0800)
committerzhishun.zhou <zhishun.zhou@samsung.com>
Fri, 20 Sep 2024 06:00:55 +0000 (14:00 +0800)
Patch from:
https://review.tizen.org/gerrit/#/c/313463/

Issue:
  Sometimes video resume failed
Reason:
  Because TizenRendererImpl::Suspend() doesn't invoke
  MediaPlayerBridgeCapi::Suspend() but invoke MediaPlayerBridgeCapi::Release();
  If suspend happened when MediaPlayerBridgeCapiTV is seeking,
  the seeking state will not be cleared after Release().
  Once resume event comes, the is_seeking_ flag will cause play fail.
Solution:
  Reset is_seeking_ in MediaPlayerBridgeCapi::Release()

Change-Id: I4bf297f33231377d3e568ec9d49d58f2ae69295e
Signed-off-by: zhishun.zhou <zhishun.zhou@samsung.com>
tizen_src/chromium_impl/media/filters/media_player_bridge_capi.cc

index 379d803912e72288a5d1ab8a941ce667a9802dee..b7d5ea8ace0a533490d246dfac64bc1fa4c159fc 100644 (file)
@@ -377,6 +377,7 @@ void MediaPlayerBridgeCapi::Release() {
   LOG_ID(INFO, player_id_) << "(" << static_cast<void*>(this) << ") "
                            << __func__;
   is_preparing_ = false;
+  is_seeking_ = false;
   StopCurrentTimeUpdateTimer();
   StopBufferingUpdateTimer();
   player_unset_completed_cb(player_);