Bug 1385745 Part 2 - Check if aFirstContinuation is the first frame of current frame. draft
authorLouis Chang <lochang@mozilla.com>
Mon, 14 Aug 2017 09:55:51 +0800
changeset 645680 74eb94444a8165cecccf4fb8db991ad22f658d46
parent 645643 7004ed70631b7ad281acce88f710a6776d533ded
child 645681 7307994befa5f9d1fead6afe64a3f7b76141b31a
push id73829
push userlochang@mozilla.com
push dateMon, 14 Aug 2017 02:12:16 +0000
bugs1385745
milestone57.0a1
Bug 1385745 Part 2 - Check if aFirstContinuation is the first frame of current frame. MozReview-Commit-ID: 6NxjfT7SVQD
layout/svg/nsSVGIntegrationUtils.cpp
--- a/layout/svg/nsSVGIntegrationUtils.cpp
+++ b/layout/svg/nsSVGIntegrationUtils.cpp
@@ -137,16 +137,18 @@ GetPreEffectsVisualOverflowUnion(nsIFram
   return collector.GetResult() + aFirstContinuationToUserSpace;
 }
 
 static nsRect
 GetPreEffectsVisualOverflow(nsIFrame* aFirstContinuation,
                             nsIFrame* aCurrentFrame,
                             const nsPoint& aFirstContinuationToUserSpace)
 {
+  MOZ_ASSERT(aFirstContinuation ==
+             nsLayoutUtils::FirstContinuationOrIBSplitSibling(aCurrentFrame));
   PreEffectsVisualOverflowCollector collector(aFirstContinuation,
                                               nullptr,
                                               nsRect(),
                                               false);
   // Compute overflow areas of current frame relative to aFirstContinuation:
   nsLayoutUtils::AddBoxesForFrame(aCurrentFrame, &collector);
   // Return the result in user space:
   return collector.GetResult() + aFirstContinuationToUserSpace;