summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorwolfbeast <mcwerewolf@wolfbeast.com>2020-03-11 13:01:48 +0100
committerwolfbeast <mcwerewolf@wolfbeast.com>2020-03-11 13:01:48 +0100
commit8dc520647d8e33e36fd64addcd4345ed6744cd08 (patch)
tree27ca88a1edc2ad4f4480715d64420ae98ddfaa57
parentfda8403f562657ec7ae340d8553e462592a30630 (diff)
downloadUXP-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.
-rw-r--r--layout/generic/nsFrame.cpp14
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