fixup! [MM][MSE][CAPI] Implement MSE with CAPI port.
authorumeshk.patel <umeshk.patel@samsung.com>
Wed, 14 Oct 2015 09:51:40 +0000 (15:21 +0530)
committerYoungsoo Choi <kenshin.choi@samsung.com>
Tue, 10 Jul 2018 07:55:23 +0000 (07:55 +0000)
Previously, Updating the seeking states was fulfilled asynchronously,
but this delayed state update can harm the state handling of seeking.

This patch is to fix updating the seeking state synchronously.

Original patch: http://suprem.sec.samsung.net/gerrit/#/c/52013/
Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=14835

Reviewed by: sm.venugopal, sns.park

Change-Id: I48187edfdd162e772820eb77ab12fbffc7271b2b
Signed-off-by: umeshk.patel <umeshk.patel@samsung.com>
tizen_src/chromium_impl/media/base/tizen/media_source_player_capi.cc

index ea745579d0422a043ac9e2381c713f934f6bb80b..76ed86ba9dea97fa7ac4e61aef627bc428eca4e6 100755 (executable)
@@ -24,7 +24,7 @@ const int kDurationUpdateInterval = 100;
 const double kSeekBufferingWaitInSec = 2;
 
 // Iframe time offset for demuxer.
-const double kDemuxerSeekIFrameTime = (0.5);
+const double kDemuxerSeekIFrameTime = 0.5;
 }  // namespace
 
 
@@ -335,10 +335,7 @@ void MediaSourcePlayerCapi::HandleBufferingMessage(int percent) {
 }
 
 void MediaSourcePlayerCapi::SeekComplete() {
-  task_runner_->PostTask(FROM_HERE,
-                       base::Bind(&MediaSourcePlayerCapi::UpdateSeekState,
-                       base::Unretained(this),
-                       MEDIA_SEEK_NONE));
+  UpdateSeekState(MEDIA_SEEK_NONE);
 
   // Initiate play for internal seeks.
   if (GetPlayerState() != PLAYER_STATE_PLAYING)
@@ -411,7 +408,7 @@ void MediaSourcePlayerCapi::SetRate(double rate) {
 }
 
 void MediaSourcePlayerCapi::Seek(const double time) {
-  seek_state_ = MEDIA_SEEK_DEMUXER;
+  UpdateSeekState(MEDIA_SEEK_DEMUXER);
   if (GetPlayerState() == PLAYER_STATE_PLAYING)
     Pause(true);