Bug 1312886: P1. Fix comments. r=cpearce
authorJean-Yves Avenard <jyavenard@mozilla.com>
Thu, 27 Oct 2016 10:05:21 +1100
changeset 319920 246305d4d3bb6595ec24f360bee9530a16648c33
parent 319919 2190ea5fc2d0f31aaf1bf9f522f6a0cd67300da8
child 319921 e24a86e2fb9fb0f9a8c6b2f9c60b6f51ee4c5db1
push id20749
push userryanvm@gmail.com
push dateSat, 29 Oct 2016 13:21:21 +0000
treeherderfx-team@1b170b39ed6b [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
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;