media: VideoRenderImpl: display last available frame

media/renderers/video_renderer_impl.cc

 // Copyright 2013 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/renderers/video_renderer_impl.h"
 
 #include "base/bind.h"
 #include "base/callback.h"
 #include "base/callback_helpers.h"
 #include "base/location.h"
@@ skipping 228 matching lines @@
     // the accuracy of our frame timing code. http://crbug.com/149829
     if (last_timestamp_ == kNoTimestamp()) {
       latest_paint_timestamp = base::TimeDelta::Max();
     } else {
       base::TimeDelta duration = target_paint_timestamp - last_timestamp_;
       latest_paint_timestamp = target_paint_timestamp + duration;
     }
 
     // Remain idle until we've reached our target paint window.
     if (now < target_paint_timestamp) {
-      UpdateStatsAndWait_Locked(kIdleTimeDelta);
+      UpdateStatsAndWait_Locked(target_paint_timestamp - now);

[DaleCurtis, 2015/03/24 17:25:23]

std::max(target_paint_timestamp - now, kIdleTimeDelta); ?

[DaleCurtis, 2015/03/24 17:47:57]

Whoops, yes.

[llandwerlin-old, 2015/03/24 17:52:00]

Done.

       continue;
     }
 
-    if (now > latest_paint_timestamp && drop_frames_) {
+    if (ready_frames_.size() > 1 && now > latest_paint_timestamp &&
+        drop_frames_) {
       DropNextReadyFrame_Locked();
       continue;
     }
 
     // Congratulations! You've made it past the video frame timing gauntlet.
     //
     // At this point enough time has passed that the next frame that ready for
     // rendering.
     PaintNextReadyFrame_Locked();
   }
@@ skipping 180 matching lines @@
     task_runner_->PostTask(FROM_HERE, base::Bind(statistics_cb_, statistics));
 
     frames_decoded_ = 0;
     frames_dropped_ = 0;
   }
 
   frame_available_.TimedWait(wait_duration);
 }
 
 }  // namespace media