Bug 1032014 - Remove extra call to AppendElement when generating animations; r=dbaron
authorBrian Birtles <birtles@gmail.com>
Thu, 03 Jul 2014 09:02:48 +0900
changeset 191961 31edc9350b848127745dfd3bb9bad3d12a3806c8
parent 191960 5206957b4f834cd82c80acedcc37a98174ac3f2c
child 191962 1b2923e5a2f78521aefa691008e9b2a0e63f86bb
push id45719
push userbbirtles@mozilla.com
push dateThu, 03 Jul 2014 00:05:27 +0000
treeherdermozilla-inbound@81ebf5784ed9 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersdbaron
bugs1032014
milestone33.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 1032014 - Remove extra call to AppendElement when generating animations; r=dbaron
layout/style/nsAnimationManager.cpp
--- a/layout/style/nsAnimationManager.cpp
+++ b/layout/style/nsAnimationManager.cpp
@@ -557,18 +557,16 @@ nsAnimationManager::BuildAnimations(nsSt
       // clear if this is the right thing to do -- we could run some of
       // the segments, but it's really not clear whether we should skip
       // values (which?) or skip segments, so best to skip the whole
       // thing for now.)
       if (!interpolated) {
         dest->mProperties.RemoveElementAt(dest->mProperties.Length() - 1);
       }
     }
-
-    aAnimations.AppendElement(dest);
   }
 }
 
 bool
 nsAnimationManager::BuildSegment(InfallibleTArray<AnimationPropertySegment>&
                                    aSegments,
                                  nsCSSProperty aProperty,
                                  const StyleAnimation& aAnimation,