Bug 1342957 - Fix incorrect MOZ_ASSERT in AnimationEffectReadOnly. r=hiro
authorBoris Chiou <boris.chiou@gmail.com>
Tue, 28 Feb 2017 01:38:52 +0800
changeset 374149 682ee6f4802745005eccdb88f831828051ae4c1b
parent 374148 3dc5e735df4f967e09af426ffcfc84e830732fce
child 374176 1bc2ad020aee2830e0a7941f10958dbec108c254
child 374177 ace3041492115096392a36c13240e3c0111eed47
push id10863
push userjlorenzo@mozilla.com
push dateMon, 06 Mar 2017 23:02:23 +0000
treeherdermozilla-aurora@0931190cd725 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewershiro
bugs1342957
milestone54.0a1
Bug 1342957 - Fix incorrect MOZ_ASSERT in AnimationEffectReadOnly. r=hiro MozReview-Commit-ID: 6amRZ9AGudQ
dom/animation/AnimationEffectReadOnly.cpp
--- a/dom/animation/AnimationEffectReadOnly.cpp
+++ b/dom/animation/AnimationEffectReadOnly.cpp
@@ -229,17 +229,17 @@ AnimationEffectReadOnly::GetComputedTimi
       break;
     case PlaybackDirection::Alternate:
       thisIterationReverse = (result.mCurrentIteration & 1) == 1;
       break;
     case PlaybackDirection::Alternate_reverse:
       thisIterationReverse = (result.mCurrentIteration & 1) == 0;
       break;
     default:
-      MOZ_ASSERT(true, "Unknown PlaybackDirection type");
+      MOZ_ASSERT_UNREACHABLE("Unknown PlaybackDirection type");
   }
   if (thisIterationReverse) {
     progress = 1.0 - progress;
   }
 
   // Calculate the 'before flag' which we use when applying step timing
   // functions.
   if ((result.mPhase == ComputedTiming::AnimationPhase::After &&