diff options
author | wolfbeast <mcwerewolf@wolfbeast.com> | 2020-03-11 13:01:48 +0100 |
---|---|---|
committer | wolfbeast <mcwerewolf@wolfbeast.com> | 2020-03-11 13:01:48 +0100 |
commit | 8dc520647d8e33e36fd64addcd4345ed6744cd08 (patch) | |
tree | 27ca88a1edc2ad4f4480715d64420ae98ddfaa57 /layout/generic/nsFrame.cpp | |
parent | fda8403f562657ec7ae340d8553e462592a30630 (diff) | |
download | UXP-8dc520647d8e33e36fd64addcd4345ed6744cd08.tar UXP-8dc520647d8e33e36fd64addcd4345ed6744cd08.tar.gz UXP-8dc520647d8e33e36fd64addcd4345ed6744cd08.tar.lz UXP-8dc520647d8e33e36fd64addcd4345ed6744cd08.tar.xz UXP-8dc520647d8e33e36fd64addcd4345ed6744cd08.zip |
Issue #1480 - Skip abspos, fixed, float and placeholders for outline.
Diffstat (limited to 'layout/generic/nsFrame.cpp')
-rw-r--r-- | layout/generic/nsFrame.cpp | 14 |
1 files changed, 12 insertions, 2 deletions
diff --git a/layout/generic/nsFrame.cpp b/layout/generic/nsFrame.cpp index 8d4ea8754..0d0c7108c 100644 --- a/layout/generic/nsFrame.cpp +++ b/layout/generic/nsFrame.cpp @@ -8447,9 +8447,13 @@ UnionBorderBoxes(nsIFrame* aFrame, bool aApplyTransform, Maybe<nsRect> clipPropClipRect = aFrame->GetClipPropClipRect(disp, effects, bounds.Size()); - // Iterate over all children except pop-ups. + // Iterate over all children except pop-ups, absolutely positioned children, + // fixed-positioned children and floats. const nsIFrame::ChildListIDs skip(nsIFrame::kPopupList | - nsIFrame::kSelectPopupList); + nsIFrame::kSelectPopupList | + nsIFrame::kAbsoluteList | + nsIFrame::kFixedList | + nsIFrame::kFloatList); for (nsIFrame::ChildListIterator childLists(aFrame); !childLists.IsDone(); childLists.Next()) { if (skip.Contains(childLists.CurrentID())) { @@ -8459,6 +8463,12 @@ UnionBorderBoxes(nsIFrame* aFrame, bool aApplyTransform, nsFrameList children = childLists.CurrentList(); for (nsFrameList::Enumerator e(children); !e.AtEnd(); e.Next()) { nsIFrame* child = e.get(); + + if (child->GetType() == nsGkAtoms::placeholderFrame) { + // Skip placeholders too. + continue; + } + // Note that passing |true| for aApplyTransform when // child->Combines3DTransformWithAncestors() is incorrect if our // aApplyTransform is false... but the opposite would be as |