cc: Video layers without a frame should not occlude

media/blink/video_frame_compositor.cc

 // Copyright 2014 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/blink/video_frame_compositor.h"
 
 #include "base/bind.h"
 #include "base/message_loop/message_loop.h"
 #include "base/time/default_tick_clock.h"
 #include "base/trace_event/trace_event.h"
@@ skipping 106 matching lines @@
   DCHECK(compositor_task_runner_->BelongsToCurrentThread());
   rendered_last_frame_ = true;
 }
 
 bool VideoFrameCompositor::UpdateCurrentFrame(base::TimeTicks deadline_min,
                                               base::TimeTicks deadline_max) {
   DCHECK(compositor_task_runner_->BelongsToCurrentThread());
   return CallRender(deadline_min, deadline_max, false);
 }
 
+bool VideoFrameCompositor::HasCurrentFrame() const {
+  return current_frame_;

[danakj, 2015/05/18 18:15:45]

can you DCHECK the thread here also?

[hendrikw, 2015/05/18 18:33:59]

Good catch, Done.

+}
+
 void VideoFrameCompositor::Start(RenderCallback* callback) {
   TRACE_EVENT0("media", "VideoFrameCompositor::Start");
 
   // Called from the media thread, so acquire the callback under lock before
   // returning in case a Stop() call comes in before the PostTask is processed.
   base::AutoLock lock(lock_);
   DCHECK(!callback_);
   callback_ = callback;
   compositor_task_runner_->PostTask(
       FROM_HERE, base::Bind(&VideoFrameCompositor::OnRendererStateUpdate,
@@ skipping 109 matching lines @@
   last_interval_ = deadline_max - deadline_min;
 
   // Restart the background rendering timer whether we're background rendering
   // or not; in either case we should wait for |kBackgroundRenderingTimeoutMs|.
   if (background_rendering_enabled_)
     background_rendering_timer_.Reset();
   return new_frame;
 }
 
 }  // namespace media