Skip to content

Commit d401966

Browse files
author
Butkovits Atila
committed
Merge mozilla-central to autoland. CLOSED TREE
2 parents a114507 + 3b59c73 commit d401966

File tree

2 files changed

+31
-33
lines changed

2 files changed

+31
-33
lines changed

layout/tables/nsTableWrapperFrame.cpp

Lines changed: 26 additions & 33 deletions
Original file line numberDiff line numberDiff line change
@@ -595,42 +595,35 @@ void nsTableWrapperFrame::CreateReflowInputForInnerTable(
595595
// sufficient to call the standard ReflowInput constructor.
596596
aChildRI.emplace(aPresContext, aOuterRI, aTableFrame, availSize, cbSize,
597597
ReflowInput::InitFlags{}, StyleSizeOverrides{}, csFlags);
598-
} else {
599-
Maybe<LogicalMargin> borderPadding;
600-
Maybe<LogicalMargin> padding;
601-
{
602-
// Compute inner table frame's border & padding because we may need to
603-
// reduce the size for inner table's size overrides. We won't waste time
604-
// if they are not used, because we can use them directly by passing them
605-
// into ReflowInput::Init().
606-
Maybe<LogicalMargin> collapseBorder;
607-
Maybe<LogicalMargin> collapsePadding;
608-
aTableFrame->GetCollapsedBorderPadding(collapseBorder, collapsePadding);
609-
SizeComputationInput input(aTableFrame, aOuterRI.mRenderingContext, wm,
610-
cbSize->ISize(wm), collapseBorder,
611-
collapsePadding);
612-
borderPadding.emplace(input.ComputedLogicalBorderPadding(wm));
613-
padding.emplace(input.ComputedLogicalPadding(wm));
614-
}
615-
616-
StyleSizeOverrides innerOverrides = ComputeSizeOverridesForInnerTable(
617-
aTableFrame, aOuterRI.mStyleSizeOverrides, borderPadding->Size(wm),
618-
aBSizeOccupiedByCaption);
619-
620-
aChildRI.emplace(aPresContext, aOuterRI, aTableFrame, availSize, Nothing(),
621-
ReflowInput::InitFlag::CallerWillInit, innerOverrides,
622-
csFlags);
623-
aChildRI->Init(aPresContext, cbSize, Some(*borderPadding - *padding),
624-
padding);
598+
return;
625599
}
626600

627-
// Carry BResize flags from aOuterRI to aChildRI.
628-
if (aOuterRI.IsBResizeForWM(wm)) {
629-
aChildRI->SetBResize(true);
630-
}
631-
if (aOuterRI.IsBResizeForPercentagesForWM(wm)) {
632-
aChildRI->SetBResizeForPercentages(true);
601+
Maybe<LogicalMargin> borderPadding;
602+
Maybe<LogicalMargin> padding;
603+
{
604+
// Compute inner table frame's border & padding because we may need to
605+
// reduce the size for inner table's size overrides. We won't waste time if
606+
// they are not used, because we can use them directly by passing them into
607+
// ReflowInput::Init().
608+
Maybe<LogicalMargin> collapseBorder;
609+
Maybe<LogicalMargin> collapsePadding;
610+
aTableFrame->GetCollapsedBorderPadding(collapseBorder, collapsePadding);
611+
SizeComputationInput input(aTableFrame, aOuterRI.mRenderingContext, wm,
612+
cbSize->ISize(wm), collapseBorder,
613+
collapsePadding);
614+
borderPadding.emplace(input.ComputedLogicalBorderPadding(wm));
615+
padding.emplace(input.ComputedLogicalPadding(wm));
633616
}
617+
618+
StyleSizeOverrides innerOverrides = ComputeSizeOverridesForInnerTable(
619+
aTableFrame, aOuterRI.mStyleSizeOverrides, borderPadding->Size(wm),
620+
aBSizeOccupiedByCaption);
621+
622+
aChildRI.emplace(aPresContext, aOuterRI, aTableFrame, availSize, Nothing(),
623+
ReflowInput::InitFlag::CallerWillInit, innerOverrides,
624+
csFlags);
625+
aChildRI->Init(aPresContext, cbSize, Some(*borderPadding - *padding),
626+
padding);
634627
}
635628

636629
void nsTableWrapperFrame::CreateReflowInputForCaption(
Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
[table-grid-item-dynamic-004.html]
2+
bug: https://bugzilla.mozilla.org/show_bug.cgi?id=1521088
3+
expected:
4+
if os == "android": [FAIL, PASS]
5+
FAIL

0 commit comments

Comments
 (0)