Enforce Stop() is always called before dtor in FFmepgVideoDecoder.

media/base/video_decoder.h

Index: media/base/video_decoder.h
diff --git a/media/base/video_decoder.h b/media/base/video_decoder.h
index b22b14438d7c7add198a778cbc0aaf1443f9454f..091cc052846ffe8ee575c0607ecba214a957bc7c 100644
--- a/media/base/video_decoder.h
+++ b/media/base/video_decoder.h
@@ -55,8 +55,10 @@ class MEDIA_EXPORT VideoDecoder
   // clean state.
   virtual void Reset(const base::Closure& closure) = 0;
 
-  // Stop decoder and set it to an uninitialized state. Note that a VideoDecoder
-  // should/could not be re-initialized after it has been stopped.
+  // Stops decoder and sets it to an uninitialized state. Note that a
+  // VideoDecoder cannot be re-initialized after it has been stopped.
+  // If the VideoDecoder has been initialized, Stop() must be called before the

[scherkus (not reviewing), 2012/09/25 20:58:16]

/must be called/must complete/

[scherkus (not reviewing), 2012/09/25 20:58:16]

s/VideoDecoder/decoder/

[Ami GONE FROM CHROMIUM, 2012/09/25 21:18:26]

Initialize() is async; "has been initialized" is unknowable by the caller. 
Replace with
If Initialize() has been called, Stop() must be called...
?

[xhwang, 2012/09/26 01:09:58]

Done.

[xhwang, 2012/09/26 01:09:58]

Done.

+  // VideoDecoder can be destructed.

[scherkus (not reviewing), 2012/09/25 20:58:16]

s/the VideoDecoder can be destructed/deleting the decoder/

[xhwang, 2012/09/26 01:09:58]

Done.

   virtual void Stop(const base::Closure& closure) = 0;
 
   // Returns true if the output format has an alpha channel. Most formats do not