Bug 1061364 - Don't force transitions to refresh their style rule; r=dbaron
☠☠ backed out by 5329cda711c8 ☠ ☠
authorBrian Birtles <birtles@gmail.com>
Tue, 24 Mar 2015 18:13:38 +0900
changeset 265556 aaf374ba062f13185c09a044e419ee5b5438bba4
parent 265555 0d377a96684ffe46395284f7bd470dc83c0c8bae
child 265557 380fb64847762a27d7a240eb44d081bcdc247bd2
push id830
push userraliiev@mozilla.com
push dateFri, 19 Jun 2015 19:24:37 +0000
treeherdermozilla-release@932614382a68 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersdbaron
bugs1061364
milestone39.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 1061364 - Don't force transitions to refresh their style rule; r=dbaron
layout/style/AnimationCommon.cpp
--- a/layout/style/AnimationCommon.cpp
+++ b/layout/style/AnimationCommon.cpp
@@ -387,20 +387,16 @@ CommonAnimationManager::GetAnimationRule
     return nullptr;
   }
 
   RestyleManager* restyleManager = mPresContext->RestyleManager();
   if (restyleManager->SkipAnimationRules()) {
     return nullptr;
   }
 
-  // FIXME: Remove this assignment.  See bug 1061364.
-  if (!IsAnimationManager()) {
-    collection->mNeedsRefreshes = true;
-  }
   collection->EnsureStyleRuleFor(
     mPresContext->RefreshDriver()->MostRecentRefresh(),
     EnsureStyleRule_IsNotThrottled);
 
   return collection->mStyleRule;
 }
 
 /* static */ const CommonAnimationManager::LayerAnimationRecord