Bug 1329098. Part 3 - remove some temp vairables. draft
authorJW Wang <jwwang@mozilla.com>
Fri, 06 Jan 2017 15:36:38 +0800
changeset 457497 71b221157db244a74f987a36a3e926be1f4fbe80
parent 457496 ce248528ce1b28c6d64bed9e0c824850837b6be8
child 457498 44023d930e2cceb9cb7bfefd254ce57e49d3e5e6
push id40791
push userjwwang@mozilla.com
push dateMon, 09 Jan 2017 03:31:36 +0000
bugs1329098
milestone53.0a1
Bug 1329098. Part 3 - remove some temp vairables. MozReview-Commit-ID: 3gz79Tzidtf
dom/media/MediaDecoderStateMachine.cpp
--- a/dom/media/MediaDecoderStateMachine.cpp
+++ b/dom/media/MediaDecoderStateMachine.cpp
@@ -2213,29 +2213,23 @@ NextFrameSeekingState::HandleVideoNotDec
       mMaster->DecodeError(aError);
   }
 }
 
 void
 MediaDecoderStateMachine::
 BufferingState::DispatchDecodeTasksIfNeeded()
 {
-  const bool needToDecodeAudio =
-    mMaster->IsAudioDecoding() &&
-   !mMaster->HaveEnoughDecodedAudio();
-
-  const bool needToDecodeVideo =
-    mMaster->IsVideoDecoding() &&
-    !mMaster->HaveEnoughDecodedVideo();
-
-  if (needToDecodeAudio) {
+  if (mMaster->IsAudioDecoding() &&
+      !mMaster->HaveEnoughDecodedAudio()) {
     mMaster->EnsureAudioDecodeTaskQueued();
   }
 
-  if (needToDecodeVideo) {
+  if (mMaster->IsVideoDecoding() &&
+      !mMaster->HaveEnoughDecodedVideo()) {
     mMaster->EnsureVideoDecodeTaskQueued();
   }
 }
 
 void
 MediaDecoderStateMachine::
 BufferingState::Step()
 {