From f8fdaf03f2b53405ab44ab2c8edcb8262f3875ee Mon Sep 17 00:00:00 2001 From: "L. David Baron" Date: Thu, 2 Oct 2014 21:53:24 -0700 Subject: [PATCH] Bug 1075137 patch 7 - Remove old IsProcessingRestyles and IsProcessingAnimationStyleChange booleans. r=birtles --- layout/base/RestyleManager.cpp | 6 ------ layout/base/nsPresContext.h | 23 ----------------------- 2 files changed, 29 deletions(-) diff --git a/layout/base/RestyleManager.cpp b/layout/base/RestyleManager.cpp index 940ff5c1fb81..2e82f4c386a1 100644 --- a/layout/base/RestyleManager.cpp +++ b/layout/base/RestyleManager.cpp @@ -1440,7 +1440,6 @@ RestyleManager::RebuildAllStyleData(nsChangeHint aExtraHint) #ifdef DEBUG mIsProcessingRestyles = true; #endif - mPresContext->SetProcessingRestyles(true); // Until we get rid of these phases in bug 960465, we need to skip // animation restyles during the non-animation phase, and post @@ -1464,7 +1463,6 @@ RestyleManager::RebuildAllStyleData(nsChangeHint aExtraHint) #ifdef DEBUG mIsProcessingRestyles = false; #endif - mPresContext->SetProcessingRestyles(false); // Make sure that we process any pending animation restyles from the // above style change. Note that we can *almost* implement the above @@ -1538,7 +1536,6 @@ RestyleManager::ProcessPendingRestyles() #ifdef DEBUG mIsProcessingRestyles = true; #endif - mPresContext->SetProcessingRestyles(true); // Before we process any restyles, we need to ensure that style // resulting from any throttled animations (animations that we're @@ -1575,15 +1572,12 @@ RestyleManager::ProcessPendingRestyles() // mid-transition (since processing the non-animation restyle ignores // the running transition so it can check for a new change on the same // property, and then posts an immediate animation style change). - mPresContext->SetProcessingAnimationStyleChange(true); MOZ_ASSERT(!mIsProcessingAnimationStyleChange, "nesting forbidden"); mIsProcessingAnimationStyleChange = true; mPendingAnimationRestyles.ProcessRestyles(); MOZ_ASSERT(mIsProcessingAnimationStyleChange, "nesting forbidden"); mIsProcessingAnimationStyleChange = false; - mPresContext->SetProcessingAnimationStyleChange(false); - mPresContext->SetProcessingRestyles(false); #ifdef DEBUG mIsProcessingRestyles = false; #endif diff --git a/layout/base/nsPresContext.h b/layout/base/nsPresContext.h index c1273e220e28..20effeb71495 100644 --- a/layout/base/nsPresContext.h +++ b/layout/base/nsPresContext.h @@ -902,26 +902,6 @@ public: mAllInvalidated = false; } - bool IsProcessingRestyles() const { - return mProcessingRestyles; - } - - void SetProcessingRestyles(bool aProcessing) { - NS_ASSERTION(aProcessing != bool(mProcessingRestyles), - "should never nest"); - mProcessingRestyles = aProcessing; - } - - bool IsProcessingAnimationStyleChange() const { - return mProcessingAnimationStyleChange; - } - - void SetProcessingAnimationStyleChange(bool aProcessing) { - NS_ASSERTION(aProcessing != bool(mProcessingAnimationStyleChange), - "should never nest"); - mProcessingAnimationStyleChange = aProcessing; - } - /** * Returns whether there are any pending restyles or reflows. */ @@ -1359,9 +1339,6 @@ protected: unsigned mIsVisual : 1; - unsigned mProcessingRestyles : 1; - unsigned mProcessingAnimationStyleChange : 1; - unsigned mFireAfterPaintEvents : 1; unsigned mIsChrome : 1;