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 345122 682ee6f4802745005eccdb88f831828051ae4c1b
parent 345121 3dc5e735df4f967e09af426ffcfc84e830732fce
child 345123 ace3041492115096392a36c13240e3c0111eed47
child 345158 1bc2ad020aee2830e0a7941f10958dbec108c254
push id38064
push userbchiou@mozilla.com
push dateTue, 28 Feb 2017 04:12:40 +0000
treeherderautoland@682ee6f48027 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewershiro
bugs1342957
milestone54.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
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 &&