Bug 1340992 - Demote MOZ_ASSERT to NS_ASSERTION before root cause addressed. r=me
authorcku <cku@mozilla.com>
Tue, 21 Feb 2017 17:44:22 +0800
changeset 373075 9d5a1af816e9e5b4090e40483f6b9189762a573f
parent 372983 e3c55cc9ffa36a578224bbcdd0c4b2c8a886908a
child 373076 4bccce8de17eda58afc4fb931685699685cf9258
push id10863
push userjlorenzo@mozilla.com
push dateMon, 06 Mar 2017 23:02:23 +0000
treeherdermozilla-aurora@0931190cd725 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersme
bugs1340992
milestone54.0a1
Bug 1340992 - Demote MOZ_ASSERT to NS_ASSERTION before root cause addressed. r=me MozReview-Commit-ID: I8goOZdPs9d
layout/svg/nsSVGIntegrationUtils.cpp
--- a/layout/svg/nsSVGIntegrationUtils.cpp
+++ b/layout/svg/nsSVGIntegrationUtils.cpp
@@ -124,20 +124,20 @@ private:
     // PreEffectsBBoxProperty might still left empty.
 #ifdef DEBUG
     if (aCheckPropCache) {
       nsIFrame* firstFrame =
         nsLayoutUtils::FirstContinuationOrIBSplitSibling(aFrame);
       bool mightHaveNoneSVGMask =
         nsSVGEffects::GetEffectProperties(firstFrame).MightHaveNoneSVGMask();
 
-      MOZ_ASSERT(mightHaveNoneSVGMask ||
-                 aFrame->GetParent()->StyleContext()->GetPseudo() ==
-                 nsCSSAnonBoxes::mozAnonymousBlock,
-                 "How did we getting here, then?");
+      NS_ASSERTION(mightHaveNoneSVGMask ||
+                   aFrame->GetParent()->StyleContext()->GetPseudo() ==
+                   nsCSSAnonBoxes::mozAnonymousBlock,
+                   "How did we getting here, then?");
     }
 
     bool hasEffect = nsSVGIntegrationUtils::UsingEffectsForFrame(aFrame);
     nsOverflowAreas* preTransformOverflows =
       aFrame->Properties().Get(aFrame->PreTransformOverflowAreasProperty());
     // Having PreTransformOverflowAreasProperty means GetVisualOverflowRect()
     // will return post-effect rect, which is not what we want, this function
     // intentional reports pre-effect rect. But it does not matter if there is