Bug 1287700 - Remove unused MDSM::mDispatchedEventToDecode. r=bechen draft
authorJW Wang <jwwang@mozilla.com>
Tue, 12 Jul 2016 13:58:40 +0800
changeset 389798 155511e04dddb38544f8d9a7e64da9c1a6b67821
parent 389797 7f2c3278538cb44ee903c39151830f24534ec3fc
child 525854 3c504c83d09468d23fa797d5871955163a763fa7
push id23515
push userjwwang@mozilla.com
push dateWed, 20 Jul 2016 02:52:39 +0000
reviewersbechen
bugs1287700
milestone50.0a1
Bug 1287700 - Remove unused MDSM::mDispatchedEventToDecode. r=bechen MozReview-Commit-ID: 5lUb83GWoNL
dom/media/MediaDecoderStateMachine.cpp
dom/media/MediaDecoderStateMachine.h
--- a/dom/media/MediaDecoderStateMachine.cpp
+++ b/dom/media/MediaDecoderStateMachine.cpp
@@ -247,17 +247,16 @@ MediaDecoderStateMachine::MediaDecoderSt
   mQuickBufferingLowDataThresholdUsecs(detail::QUICK_BUFFERING_LOW_DATA_USECS),
   mSuspendBackgroundVideoDelay(SuspendBackgroundVideoDelay()),
   mIsAudioPrerolling(false),
   mIsVideoPrerolling(false),
   mAudioCaptured(false),
   INIT_WATCHABLE(mAudioCompleted, false),
   INIT_WATCHABLE(mVideoCompleted, false),
   mNotifyMetadataBeforeFirstFrame(false),
-  mDispatchedEventToDecode(false),
   mQuickBuffering(false),
   mMinimizePreroll(false),
   mDecodeThreadWaiting(false),
   mDecodingFirstFrame(true),
   mSentLoadedMetadataEvent(false),
   mSentFirstFrameLoadedEvent(false),
   mSentPlaybackEndedEvent(false),
   mVideoDecodeSuspended(false),
--- a/dom/media/MediaDecoderStateMachine.h
+++ b/dom/media/MediaDecoderStateMachine.h
@@ -866,21 +866,16 @@ private:
   // Flag whether we notify metadata before decoding the first frame or after.
   //
   // Note that the odd semantics here are designed to replicate the current
   // behavior where we notify the decoder each time we come out of dormant, but
   // send suppressed event visibility for those cases. This code can probably be
   // simplified.
   bool mNotifyMetadataBeforeFirstFrame;
 
-  // True if we've dispatched an event to the decode task queue to call
-  // DecodeThreadRun(). We use this flag to prevent us from dispatching
-  // unnecessary runnables, since the decode thread runs in a loop.
-  bool mDispatchedEventToDecode;
-
   // If this is true while we're in buffering mode, we can exit early,
   // as it's likely we may be able to playback. This happens when we enter
   // buffering mode soon after the decode starts, because the decode-ahead
   // ran fast enough to exhaust all data while the download is starting up.
   // Synchronised via decoder monitor.
   bool mQuickBuffering;
 
   // True if we should not decode/preroll unnecessary samples, unless we're