Fix race condition in media Pipeline::GetMediaTime() after seeking.

media/base/pipeline_impl.cc

Index: media/base/pipeline_impl.cc
diff --git a/media/base/pipeline_impl.cc b/media/base/pipeline_impl.cc
index 85d470a022758459ac8f8f7f7bae599a088d7b76..d82547a44a20cefe3192f8776cc83e1470be0cd2 100644
--- a/media/base/pipeline_impl.cc
+++ b/media/base/pipeline_impl.cc
@@ -958,6 +958,7 @@ void PipelineImpl::Start(Demuxer* demuxer,
   client_ = client;
   seek_cb_ = seek_cb;
   last_media_time_ = base::TimeDelta();
+  seek_time_ = kNoTimestamp;
 
   std::unique_ptr<TextRenderer> text_renderer;
   if (base::CommandLine::ForCurrentProcess()->HasSwitch(
@@ -1039,6 +1040,7 @@ void PipelineImpl::Seek(base::TimeDelta time, const PipelineStatusCB& seek_cb) {
 
   DCHECK(seek_cb_.is_null());
   seek_cb_ = seek_cb;
+  seek_time_ = time;
   last_media_time_ = base::TimeDelta();
   media_task_runner_->PostTask(
       FROM_HERE, base::Bind(&RendererWrapper::Seek,
@@ -1069,6 +1071,7 @@ void PipelineImpl::Resume(std::unique_ptr<Renderer> renderer,
   DCHECK(IsRunning());
   DCHECK(seek_cb_.is_null());
   seek_cb_ = seek_cb;
+  seek_time_ = time;
   last_media_time_ = base::TimeDelta();
 
   media_task_runner_->PostTask(
@@ -1123,7 +1126,11 @@ void PipelineImpl::SetVolume(float volume) {
 base::TimeDelta PipelineImpl::GetMediaTime() const {
   DCHECK(thread_checker_.CalledOnValidThread());
 
-  base::TimeDelta media_time = renderer_wrapper_->GetMediaTime();
+  // Don't trust renderer time during a pending seek. Renderer may return
+  // pre-seek time which may corrupt |last_media_time_| used for clamping.
+  base::TimeDelta media_time = seek_time_ != kNoTimestamp
+                                   ? seek_time_
+                                   : renderer_wrapper_->GetMediaTime();
 
   // Clamp current media time to the last reported value, this prevents higher
   // level clients from seeing time go backwards based on inaccurate or spurious
@@ -1301,6 +1308,8 @@ void PipelineImpl::OnSeekDone() {
   DCHECK(thread_checker_.CalledOnValidThread());
   DCHECK(IsRunning());
 
+  seek_time_ = kNoTimestamp;
+
   DCHECK(!seek_cb_.is_null());
   base::ResetAndReturn(&seek_cb_).Run(PIPELINE_OK);
 }