Interpolate media time for mojo rendering pipeline.

media/base/pipeline_impl.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "media/base/pipeline_impl.h"
 
 #include <algorithm>
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
@@ skipping 884 matching lines @@
   DCHECK(thread_checker_.CalledOnValidThread());
   DCHECK(demuxer);
   DCHECK(renderer);
   DCHECK(client);
   DCHECK(!seek_cb.is_null());
 
   DCHECK(!client_);
   DCHECK(seek_cb_.is_null());
   client_ = client;
   seek_cb_ = seek_cb;
+  last_media_time_ = base::TimeDelta();
 
   std::unique_ptr<TextRenderer> text_renderer;
   if (base::CommandLine::ForCurrentProcess()->HasSwitch(
           switches::kEnableInbandTextTracks)) {
     text_renderer.reset(new TextRenderer(
         media_task_runner_,
         BindToCurrentLoop(base::Bind(&PipelineImpl::OnAddTextTrack,
                                      weak_factory_.GetWeakPtr()))));
   }
 
@@ skipping 61 matching lines @@
   DCHECK(thread_checker_.CalledOnValidThread());
   DCHECK(!seek_cb.is_null());
 
   if (!IsRunning()) {
     DLOG(ERROR) << "Media pipeline isn't running. Ignoring Seek().";
     return;
   }
 
   DCHECK(seek_cb_.is_null());
   seek_cb_ = seek_cb;
+  last_media_time_ = base::TimeDelta();
   media_task_runner_->PostTask(
       FROM_HERE, base::Bind(&RendererWrapper::Seek,
                             base::Unretained(renderer_wrapper_.get()), time));
 }
 
 void PipelineImpl::Suspend(const PipelineStatusCB& suspend_cb) {
   DVLOG(2) << __func__;
   DCHECK(!suspend_cb.is_null());
 
   DCHECK(IsRunning());
   DCHECK(suspend_cb_.is_null());
   suspend_cb_ = suspend_cb;
 
   media_task_runner_->PostTask(
       FROM_HERE, base::Bind(&RendererWrapper::Suspend,
                             base::Unretained(renderer_wrapper_.get())));
 }
 
 void PipelineImpl::Resume(std::unique_ptr<Renderer> renderer,
                           base::TimeDelta time,
                           const PipelineStatusCB& seek_cb) {
   DVLOG(2) << __func__;
   DCHECK(thread_checker_.CalledOnValidThread());
   DCHECK(renderer);
   DCHECK(!seek_cb.is_null());
 
   DCHECK(IsRunning());
   DCHECK(seek_cb_.is_null());
   seek_cb_ = seek_cb;
+  last_media_time_ = base::TimeDelta();
 
   media_task_runner_->PostTask(
       FROM_HERE, base::Bind(&RendererWrapper::Resume,
                             base::Unretained(renderer_wrapper_.get()),
                             base::Passed(&renderer), time));
 }
 
 bool PipelineImpl::IsRunning() const {
   DCHECK(thread_checker_.CalledOnValidThread());
   return !!client_;
@@ skipping 32 matching lines @@
 
   volume_ = volume;
   media_task_runner_->PostTask(
       FROM_HERE,
       base::Bind(&RendererWrapper::SetVolume,
                  base::Unretained(renderer_wrapper_.get()), volume_));
 }
 
 base::TimeDelta PipelineImpl::GetMediaTime() const {
   DCHECK(thread_checker_.CalledOnValidThread());
-  return renderer_wrapper_->GetMediaTime();
+
+  base::TimeDelta media_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
+  // delay values reported to the AudioClock.
+  //
+  // It is expected that such events are transient and will be recovered as
+  // rendering continues over time.
+  if (media_time < last_media_time_) {
+    DVLOG(2) << __func__ << ": actual=" << media_time
+             << " clamped=" << last_media_time_;
+    return last_media_time_;
+  }
+
+  DVLOG(3) << __FUNCTION__ << ": " << media_time.InMilliseconds() << " ms";
+  last_media_time_ = media_time;
+  return last_media_time_;
 }
 
 Ranges<base::TimeDelta> PipelineImpl::GetBufferedTimeRanges() const {
   DCHECK(thread_checker_.CalledOnValidThread());
   return renderer_wrapper_->GetBufferedTimeRanges();
 }
 
 base::TimeDelta PipelineImpl::GetMediaDuration() const {
   DCHECK(thread_checker_.CalledOnValidThread());
   return duration_;
@@ skipping 155 matching lines @@
 void PipelineImpl::OnSuspendDone() {
   DVLOG(3) << __func__;
   DCHECK(thread_checker_.CalledOnValidThread());
   DCHECK(IsRunning());
 
   DCHECK(!suspend_cb_.is_null());
   base::ResetAndReturn(&suspend_cb_).Run(PIPELINE_OK);
 }
 
 }  // namespace media