[Video] Reset new processed frame callback on the compositor thread.

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/callback_helpers.h"
 #include "base/message_loop/message_loop.h"
 #include "base/time/default_tick_clock.h"
@@ skipping 45 matching lines @@
 
   if (!auto_open_close_) {
     auto_open_close_.reset(new base::trace_event::AutoOpenCloseEvent(
         base::trace_event::AutoOpenCloseEvent::Type::ASYNC, "media,rail",
         "VideoPlayback"));
   }
 
   if (rendering_) {
     auto_open_close_->Begin();
   } else {
+    new_processed_frame_cb_.Reset();
     auto_open_close_->End();
   }
 
   if (rendering_) {
     // Always start playback in background rendering mode, if |client_| kicks
     // in right away it's okay.
     BackgroundRender();
   } else if (background_rendering_enabled_) {
     background_rendering_timer_.Stop();
   } else {
@@ skipping 56 matching lines @@
 void VideoFrameCompositor::Stop() {
   // Called from the media thread, so release the callback under lock before
   // returning to avoid a pending UpdateCurrentFrame() call occurring before
   // the PostTask is processed.
   base::AutoLock lock(callback_lock_);
   DCHECK(callback_);
   callback_ = nullptr;
   compositor_task_runner_->PostTask(
       FROM_HERE, base::Bind(&VideoFrameCompositor::OnRendererStateUpdate,
                             base::Unretained(this), false));
-  new_processed_frame_cb_.Reset();
 }
 
 void VideoFrameCompositor::PaintSingleFrame(
     const scoped_refptr<VideoFrame>& frame,
     bool repaint_duplicate_frame) {
   if (!compositor_task_runner_->BelongsToCurrentThread()) {
     compositor_task_runner_->PostTask(
         FROM_HERE,
         base::Bind(&VideoFrameCompositor::PaintSingleFrame,
                    base::Unretained(this), frame, repaint_duplicate_frame));
@@ 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 || had_new_background_frame;
 }
 
 }  // namespace media