Fix race condition when accessing time_progressing_.

media/renderers/video_renderer_impl.h

 // 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.
 
 #ifndef MEDIA_RENDERERS_VIDEO_RENDERER_IMPL_H_
 #define MEDIA_RENDERERS_VIDEO_RENDERER_IMPL_H_
 
 #include <deque>
 
 #include "base/memory/ref_counted.h"
@@ skipping 129 matching lines @@
 
   // Returns true if there is no more room for additional buffered frames.
   bool HaveReachedBufferingCap();
 
   // Starts or stops |sink_| respectively. Do not call while |lock_| is held.
   void StartSink();
   void StopSink();
 
   // Fires |ended_cb_| if there are no remaining usable frames and
   // |received_end_of_stream_| is true.  Sets |rendered_end_of_stream_| if it
-  // does so.  Returns algorithm_->EffectiveFramesQueued().
-  size_t MaybeFireEndedCallback();
+  // does so.
+  //
+  // When called from the media thread, |time_progressing| should reflect the
+  // value of |time_progressing_|.  When called from Render() on the sink
+  // callback thread, the inverse of |render_first_frame_and_stop_| should be
+  // used as a proxy for |time_progressing_|.
+  //
+  // Returns algorithm_->EffectiveFramesQueued().
+  size_t MaybeFireEndedCallback_Locked(bool time_progressing);
 
   // Helper method for converting a single media timestamp to wall clock time.
   base::TimeTicks ConvertMediaTimestamp(base::TimeDelta media_timestamp);
 
   scoped_refptr<base::SingleThreadTaskRunner> task_runner_;
 
   // Enables the use of VideoRendererAlgorithm and VideoRendererSink for frame
   // rendering instead of using a thread in a sleep-loop.  Set via the command
   // line flag kEnableNewVideoRenderer or via test methods.
   bool use_new_video_renderering_path_;
 
   // Sink which calls into VideoRendererImpl via Render() for video frames.  Do
   // not call any methods on the sink while |lock_| is held or the two threads
   // might deadlock. Do not call Start() or Stop() on the sink directly, use
-  // StartSink() and StopSink() to ensure background rendering is started.
+  // StartSink() and StopSink() to ensure background rendering is started.  Only
+  // access these values on |task_runner_|.
   VideoRendererSink* const sink_;
   bool sink_started_;
 
   // Used for accessing data members.
   base::Lock lock_;
 
   // Provides video frames to VideoRendererImpl.
   scoped_ptr<VideoFrameStream> video_frame_stream_;
 
   // Pool of GpuMemoryBuffers and resources used to create hardware frames.
@@ skipping 87 matching lines @@
 
   // Algorithm for selecting which frame to render; manages frames and all
   // timing related information.
   scoped_ptr<VideoRendererAlgorithm> algorithm_;
 
   // Indicates that Render() was called with |background_rendering| set to true,
   // so we've entered a background rendering mode where dropped frames are not
   // counted.  Must be accessed under |lock_| once |sink_| is started.
   bool was_background_rendering_;
 
-  // Indicates whether or not media time is currently progressing or not.
+  // Indicates whether or not media time is currently progressing or not.  Must
+  // only be accessed from |task_runner_|.
   bool time_progressing_;
 
   // Indicates that Render() should only render the first frame and then request
   // that the sink be stopped.  |posted_maybe_stop_after_first_paint_| is used
   // to avoid repeated task posts.
   bool render_first_frame_and_stop_;
   bool posted_maybe_stop_after_first_paint_;
 
   // NOTE: Weak pointers must be invalidated before all other member variables.
   base::WeakPtrFactory<VideoRendererImpl> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(VideoRendererImpl);
 };
 
 }  // namespace media
 
 #endif  // MEDIA_RENDERERS_VIDEO_RENDERER_IMPL_H_