Bug 1283370. Part 4 - remove HasAudio(). r=kaku draft
authorJW Wang <jwwang@mozilla.com>
Thu, 30 Jun 2016 17:42:57 +0800
changeset 384376 d5e41b7fac5053ac36a1178a2f8e29eb077ada65
parent 384375 e3ee4f932e744a5385d8dc8251a99db3ff2c293b
child 384377 c2775fdc05e8557f098363bd682881642c7f183f
push id22256
push userjwwang@mozilla.com
push dateWed, 06 Jul 2016 06:43:32 +0000
reviewerskaku
bugs1283370
milestone50.0a1
Bug 1283370. Part 4 - remove HasAudio(). r=kaku We guarantee to have |!mReader->IsRequestingAudioData() && !mReader->IsWaitingAudioData()| when there is no audio at all. We can remove the check for HasAudio(). MozReview-Commit-ID: KYYIq6n06Vr
dom/media/NextFrameSeekTask.cpp
dom/media/NextFrameSeekTask.h
--- a/dom/media/NextFrameSeekTask.cpp
+++ b/dom/media/NextFrameSeekTask.cpp
@@ -45,39 +45,31 @@ NextFrameSeekTask::NextFrameSeekTask(con
                                    const media::TimeUnit& aDuration,
                                    int64_t aCurrentMediaTime,
                                    MediaQueue<MediaData>& aAudioQueue,
                                    MediaQueue<MediaData>& aVideoQueue)
   : SeekTask(aDecoderID, aThread, aReader, Move(aSeekJob))
   , mAudioQueue(aAudioQueue)
   , mVideoQueue(aVideoQueue)
   , mCurrentTimeBeforeSeek(aCurrentMediaTime)
-  , mHasAudio(aInfo.HasAudio())
   , mDuration(aDuration)
 {
   AssertOwnerThread();
   MOZ_ASSERT(aInfo.HasVideo());
 
   // Configure MediaDecoderReaderWrapper.
   SetCallbacks();
 }
 
 NextFrameSeekTask::~NextFrameSeekTask()
 {
   AssertOwnerThread();
   MOZ_ASSERT(mIsDiscarded);
 }
 
-bool
-NextFrameSeekTask::HasAudio() const
-{
-  AssertOwnerThread();
-  return mHasAudio;
-}
-
 void
 NextFrameSeekTask::Discard()
 {
   AssertOwnerThread();
 
   // Disconnect MediaDecoder.
   mSeekJob.RejectIfExists(__func__);
 
@@ -235,18 +227,17 @@ NextFrameSeekTask::IsAudioSeekComplete()
 {
   AssertOwnerThread();
   SAMPLE_LOG("IsAudioSeekComplete() curTarVal=%d aqFin=%d aqSz=%d req=%d wait=%d",
     mSeekJob.Exists(), mIsAudioQueueFinished, !!mSeekedAudioData,
     mReader->IsRequestingAudioData(), mReader->IsWaitingAudioData());
 
   // Just make sure that we are not requesting or waiting for audio data. We
   // don't really need to get an decoded audio data or get EOS here.
-  return !HasAudio() ||
-         (!mReader->IsRequestingAudioData() && !mReader->IsWaitingAudioData());
+  return !mReader->IsRequestingAudioData() && !mReader->IsWaitingAudioData();
 }
 
 bool
 NextFrameSeekTask::IsVideoSeekComplete()
 {
   AssertOwnerThread();
   SAMPLE_LOG("IsVideoSeekComplete() curTarVal=%d vqFin=%d vqSz=%d",
       mSeekJob.Exists(), mIsVideoQueueFinished, !!mSeekedVideoData);
--- a/dom/media/NextFrameSeekTask.h
+++ b/dom/media/NextFrameSeekTask.h
@@ -38,18 +38,16 @@ public:
 
   RefPtr<SeekTaskPromise> Seek(const media::TimeUnit& aDuration) override;
 
   bool NeedToResetMDSM() const override;
 
 private:
   ~NextFrameSeekTask();
 
-  bool HasAudio() const;
-
   bool IsVideoDecoding() const;
 
   nsresult EnsureVideoDecodeTaskQueued();
 
   const char* VideoRequestStatus();
 
   void RequestVideoData();
 
@@ -80,17 +78,16 @@ private:
    */
   MediaQueue<MediaData>& mAudioQueue;
   MediaQueue<MediaData>& mVideoQueue;
 
   /*
    * Internal state.
    */
   const int64_t mCurrentTimeBeforeSeek;
-  const bool mHasAudio;
   media::TimeUnit mDuration;
 
   MediaEventListener mAudioCallback;
   MediaEventListener mVideoCallback;
   MediaEventListener mAudioWaitCallback;
   MediaEventListener mVideoWaitCallback;
 };