Remove refcounting of VideoDecodeEngine

VideoDecodeEngine doesn't need refcounting, all decode engines live on one
thread so there isn't a problem posting task to a random thread. Also
the stop is asynchronous, we can always until the stop is done before we
destroy the object and doesn't need refcounting to destroy it.

TEST=media_unittests

Review URL: http://codereview.chromium.org/3125032

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@58539 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 4 insertions, 4 deletions

diff --git a/media/video/video_decode_engine.h b/media/video/video_decode_engine.h
index d2cf7b2..9c84f11 100644
--- a/media/video/video_decode_engine.h
+++ b/media/video/video_decode_engine.h
@@ -68,7 +68,7 @@ struct VideoCodecInfo {
   VideoStreamInfo stream_info_;
 };
 
-class VideoDecodeEngine : public base::RefCountedThreadSafe<VideoDecodeEngine> {
+class VideoDecodeEngine {
  public:
   struct EventHandler {
    public:
@@ -83,7 +83,6 @@ class VideoDecodeEngine : public base::RefCountedThreadSafe<VideoDecodeEngine> {
     virtual void OnFillBufferCallback(scoped_refptr<VideoFrame> frame) = 0;
   };
 
-  VideoDecodeEngine() {}
   virtual ~VideoDecodeEngine() {}
 
   // Initialized the engine with specified configuration. |message_loop| could
@@ -103,8 +102,9 @@ class VideoDecodeEngine : public base::RefCountedThreadSafe<VideoDecodeEngine> {
   // could be itself. ) then call EventHandler::OnFlushDone().
   virtual void Flush() = 0;
 
-  // Used in openmax to InitialReadBuffers().
-  virtual void Seek() = 0;  // TODO(jiesun): Do we need this?
+  // This method is used as a signal for the decode engine to prefoll and
+  // issue read requests after Flush() us made.
+  virtual void Seek() = 0;
 
   // Buffer exchange method for input and output stream.
   // These functions and callbacks could be used in two scenarios for both