From a1236de6d5283259bdef9e28b9c25d38aaa9b463 Mon Sep 17 00:00:00 2001 From: "L. David Baron" Date: Thu, 2 Oct 2014 21:53:22 -0700 Subject: [PATCH] Bug 1074634 - Don't overwrite eRestyleResult_ContinueAndForceDescendants with eRestyleResult_Continue. r=heycam --- layout/base/RestyleManager.cpp | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/layout/base/RestyleManager.cpp b/layout/base/RestyleManager.cpp index 74b53b25281d..39ebdca0c330 100644 --- a/layout/base/RestyleManager.cpp +++ b/layout/base/RestyleManager.cpp @@ -2746,9 +2746,7 @@ ElementRestyler::RestyleSelf(nsIFrame* aSelf, RestyleResult result; - if (aRestyleHint & eRestyle_ForceDescendants) { - result = eRestyleResult_ContinueAndForceDescendants; - } else if (aRestyleHint) { + if (aRestyleHint) { result = eRestyleResult_Continue; } else { result = ComputeRestyleResultFromFrame(aSelf); @@ -3158,6 +3156,10 @@ ElementRestyler::RestyleSelf(nsIFrame* aSelf, } } + if (aRestyleHint & eRestyle_ForceDescendants) { + result = eRestyleResult_ContinueAndForceDescendants; + } + LOG_RESTYLE("returning %s", RestyleResultToString(result).get()); return result;