Bug 1385748. P1 - remove MDSM::IsLogicallyPlaying() which has no callers at all. draft
authorJW Wang <jwwang@mozilla.com>
Mon, 31 Jul 2017 10:15:47 +0800
changeset 618247 a2b4673559bb7668c78aa9a6ed5d837a14803778
parent 618243 f86b01a7cbaddae32b10cbeac1df76735da1f2c7
child 618248 9de146aa3341e481bad7fab6f1d62d8b5609962e
push id71262
push userjwwang@mozilla.com
push dateMon, 31 Jul 2017 02:39:25 +0000
bugs1385748
milestone56.0a1
Bug 1385748. P1 - remove MDSM::IsLogicallyPlaying() which has no callers at all. MozReview-Commit-ID: IfpzTMVtgf6
dom/media/MediaDecoderStateMachine.h
--- a/dom/media/MediaDecoderStateMachine.h
+++ b/dom/media/MediaDecoderStateMachine.h
@@ -525,28 +525,16 @@ private:
 
   // FrameID which increments every time a frame is pushed to our queue.
   FrameID mCurrentFrameID;
 
   // The highest timestamp that our position has reached. Monotonically
   // increasing.
   Watchable<media::TimeUnit> mObservedDuration;
 
-  // Returns true if we're logically playing, that is, if the Play() has
-  // been called and Pause() has not or we have not yet reached the end
-  // of media. This is irrespective of the seeking state; if the owner
-  // calls Play() and then Seek(), we still count as logically playing.
-  // The decoder monitor must be held.
-  bool IsLogicallyPlaying()
-  {
-    MOZ_ASSERT(OnTaskQueue());
-    return mPlayState == MediaDecoder::PLAY_STATE_PLAYING
-           || mNextPlayState == MediaDecoder::PLAY_STATE_PLAYING;
-  }
-
   // Media Fragment end time.
   media::TimeUnit mFragmentEndTime = media::TimeUnit::Invalid();
 
   // The media sink resource.  Used on the state machine thread.
   RefPtr<media::MediaSink> mMediaSink;
 
   const RefPtr<ReaderProxy> mReader;