Bug 1933101 Part 1 - Use StyleSize::Size() instead of ISize() and BSize() separately. r=layout-reviewers,emilio

Differential Revision: https://phabricator.services.mozilla.com/D230063
This commit is contained in:
Ting-Yu Lin
2024-11-25 16:38:53 +00:00
parent 2a6b0151af
commit e6851290c7

View File

@@ -862,8 +862,7 @@ struct nsGridContainerFrame::GridItemInfo {
mFrame->IsTableWrapperFrame() mFrame->IsTableWrapperFrame()
? mFrame->PrincipalChildList().FirstChild()->StylePosition() ? mFrame->PrincipalChildList().FirstChild()->StylePosition()
: mFrame->StylePosition(); : mFrame->StylePosition();
const auto& size = const auto& size = pos->Size(aContainerAxis, aContainerWM);
isInlineAxis ? pos->ISize(aContainerWM) : pos->BSize(aContainerWM);
// max-content and min-content should behave as initial value in block axis. // max-content and min-content should behave as initial value in block axis.
// FIXME: Bug 567039: moz-fit-content and -moz-available are not supported // FIXME: Bug 567039: moz-fit-content and -moz-available are not supported
// for block size dimension on sizing properties (e.g. height), so we // for block size dimension on sizing properties (e.g. height), so we
@@ -878,8 +877,7 @@ struct nsGridContainerFrame::GridItemInfo {
if (!isAuto && !::IsPercentOfIndefiniteSize(size, aPercentageBasis)) { if (!isAuto && !::IsPercentOfIndefiniteSize(size, aPercentageBasis)) {
return false; return false;
} }
const auto& minSize = isInlineAxis ? pos->MinISize(aContainerWM) const auto& minSize = pos->MinSize(aContainerAxis, aContainerWM);
: pos->MinBSize(aContainerWM);
// max-content and min-content should behave as initial value in block axis. // max-content and min-content should behave as initial value in block axis.
// FIXME: Bug 567039: moz-fit-content and -moz-available are not supported // FIXME: Bug 567039: moz-fit-content and -moz-available are not supported
// for block size dimension on sizing properties (e.g. height), so we // for block size dimension on sizing properties (e.g. height), so we