Bug 1312886: P1. Fix comments. r?cpearce draft
authorJean-Yves Avenard <jyavenard@mozilla.com>
Thu, 27 Oct 2016 10:05:21 +1100
changeset 430758 a7073d5bc10324fada58e4e88e315e44cb99749f
parent 430701 944cb0fd05526894fcd90fbe7d1e625ee53cd73d
child 430759 f921259d51206b2fd4b16a8ed322cddb34de373b
push id33879
push userbmo:jyavenard@mozilla.com
push dateFri, 28 Oct 2016 03:36:01 +0000
reviewerscpearce
bugs1312886
milestone52.0a1
Bug 1312886: P1. Fix comments. r?cpearce A MediaDataDecoder now always call InputExhausted once decoding is completed. MozReview-Commit-ID: K6SCRYb2ieI
dom/media/MediaFormatReader.h
--- a/dom/media/MediaFormatReader.h
+++ b/dom/media/MediaFormatReader.h
@@ -301,21 +301,20 @@ private:
 
     // MediaDataDecoder handler's variables.
     // Decoder initialization promise holder.
     MozPromiseRequestHolder<MediaDataDecoder::InitPromise> mInitPromise;
     // False when decoder is created. True when decoder Init() promise is resolved.
     bool mDecoderInitialized;
     bool mOutputRequested;
     // Set to true once the MediaDataDecoder has been fed a compressed sample.
-    // No more sample will be passed to the decoder while true.
+    // No more samples will be passed to the decoder while true.
     // mDecodePending is reset when:
-    // 1- The decoder returns a sample
-    // 2- The decoder calls InputExhausted
-    // 3- The decoder is Flushed or Reset.
+    // 1- The decoder calls InputExhausted
+    // 2- The decoder is Flushed or Reset.
     bool mDecodePending;
     bool mNeedDraining;
     bool mDraining;
     bool mDrainComplete;
 
     bool HasPendingDrain() const
     {
       return mDraining || mDrainComplete;