From eed121aaa96fe88bef9f1a3abc8674ce7c2cdf6a Mon Sep 17 00:00:00 2001 From: "L. David Baron" Date: Tue, 13 Jan 2015 21:03:12 -0800 Subject: [PATCH] Bug 1115812 patch 14 - For rem unit change handling, repost min hint to root just like change hint. r=heycam This changes what was probably a silly design choice when I wrote the code for 'rem'-basis handling; we shouldn't try continuing through the rest of RestyleElement() here, but instead repost the hint to the rebuild-all process. --- layout/base/RestyleManager.cpp | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/layout/base/RestyleManager.cpp b/layout/base/RestyleManager.cpp index bfe681c19f29..c0a99e1ad5e3 100644 --- a/layout/base/RestyleManager.cpp +++ b/layout/base/RestyleManager.cpp @@ -951,11 +951,9 @@ RestyleManager::RestyleElement(Element* aElement, // The basis for 'rem' units has changed. newContext = nullptr; mRebuildAllRestyleHint |= aRestyleHint; + NS_UpdateHint(mRebuildAllExtraHint, aMinHint); DoRebuildAllStyleData(aRestyleTracker); - if (aMinHint == 0) { - return; - } - aPrimaryFrame = aElement->GetPrimaryFrame(); + return; } } }