Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(228)

Unified Diff: media/base/video_renderer.h

Issue 12262058: Revert r180578, r180591, and r180604 from 1410 branch. (Closed) Base URL: svn://svn.chromium.org/chrome/branches/1410/src/
Patch Set: Created 7 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « media/base/pipeline_unittest.cc ('k') | media/filters/audio_renderer_impl.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/base/video_renderer.h
===================================================================
--- media/base/video_renderer.h (revision 182591)
+++ media/base/video_renderer.h (working copy)
@@ -22,7 +22,8 @@
class DemuxerStream;
class VideoDecoder;
-class MEDIA_EXPORT VideoRenderer {
+class MEDIA_EXPORT VideoRenderer
+ : public base::RefCountedThreadSafe<VideoRenderer> {
public:
typedef std::list<scoped_refptr<VideoDecoder> > VideoDecoderList;
@@ -36,9 +37,6 @@
// Used to query the current time or duration of the media.
typedef base::Callback<base::TimeDelta()> TimeDeltaCB;
- VideoRenderer();
- virtual ~VideoRenderer();
-
// Initialize a VideoRenderer with the given DemuxerStream and
// VideoDecoderList, executing |init_cb| callback upon completion.
//
@@ -93,6 +91,12 @@
// Updates the current playback rate.
virtual void SetPlaybackRate(float playback_rate) = 0;
+ protected:
+ friend class base::RefCountedThreadSafe<VideoRenderer>;
+
+ VideoRenderer();
+ virtual ~VideoRenderer();
+
private:
DISALLOW_COPY_AND_ASSIGN(VideoRenderer);
};
« no previous file with comments | « media/base/pipeline_unittest.cc ('k') | media/filters/audio_renderer_impl.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698