GpuVideoDecoder not to use DemuxerStream after it's stopped.

media/base/video_decoder.h

 // Copyright (c) 2012 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_BASE_VIDEO_DECODER_H_
 #define MEDIA_BASE_VIDEO_DECODER_H_
 
 #include "base/callback.h"
 #include "base/memory/ref_counted.h"
 #include "media/base/pipeline_status.h"
@@ skipping 41 matching lines @@
   virtual void Read(const ReadCB& read_cb) = 0;
 
   // Resets decoder state, fulfilling all pending ReadCB and dropping extra
   // queued decoded data. After this call, the decoder is back to an initialized
   // clean state.
   // Note: No VideoDecoder calls should be made before |closure| is executed.
   virtual void Reset(const base::Closure& closure) = 0;
 
   // Stops decoder, fires any pending callbacks and sets the decoder to an
   // uninitialized state. A VideoDecoder cannot be re-initialized after it has
-  // been stopped.
+  // been stopped. A VideoDecoder should not access the DemuxerStream after it

[Ami GONE FROM CHROMIUM, 2013/04/04 03:43:22]

This seems like a commentary on the lifecycle of the demuxerstream arg to
Initialize.

[xhwang, 2013/04/04 17:05:33]

Done.

+  // has been stopped.
   // Note that if Initialize() has been called, Stop() must be called and
   // complete before deleting the decoder.
   virtual void Stop(const base::Closure& closure) = 0;
 
   // Returns true if the output format has an alpha channel. Most formats do not
   // have alpha so the default is false. Override and return true for decoders
   // that return formats with an alpha channel.
   virtual bool HasAlpha() const;
 
   // Returns true if the decoder currently has the ability to decode and return
   // a VideoFrame. Most implementations can allocate a new VideoFrame and hence
   // this will always return true. Override and return false for decoders that
   // use a fixed set of VideoFrames for decoding.
   virtual bool HasOutputFrameAvailable() const;
 
  protected:
   friend class base::RefCountedThreadSafe<VideoDecoder>;
   virtual ~VideoDecoder();
   VideoDecoder();
 
   DISALLOW_COPY_AND_ASSIGN(VideoDecoder);
 };
 
 }  // namespace media
 
 #endif  // MEDIA_BASE_VIDEO_DECODER_H_