https://bugs.webkit.org/show_bug.cgi?id=144144
Reviewed by Darin Adler.
Source/WebCore:
Change order in hasDefiniteLogicalHeight() ifs to consider definite
absolutely positioned elements.
Test: fast/css-grid-layout/percent-track-breadths-regarding-container-size.html
* rendering/RenderBox.cpp:
(WebCore::RenderBox::hasDefiniteLogicalHeight):
LayoutTests:
Add new test case to check absolutely positioned elements.
* fast/css-grid-layout/percent-track-breadths-regarding-container-size-expected.txt:
* fast/css-grid-layout/percent-track-breadths-regarding-container-size.html:
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@183385
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
+2015-04-26 Manuel Rego Casasnovas <rego@igalia.com>
+
+ [CSS Grid Layout] LayoutBox::hasDefiniteLogicalHeight() should consider abspos boxes as definite
+ https://bugs.webkit.org/show_bug.cgi?id=144144
+
+ Reviewed by Darin Adler.
+
+ Add new test case to check absolutely positioned elements.
+
+ * fast/css-grid-layout/percent-track-breadths-regarding-container-size-expected.txt:
+ * fast/css-grid-layout/percent-track-breadths-regarding-container-size.html:
+
2015-04-26 Ryosuke Niwa <rniwa@webkit.org>
Class body ending with a semicolon throws a SyntaxError
XXXXX
XXX
PASS
+XX
+XXXXX
+XXX
+PASS
+XX
+XXXXX
+XXX
+PASS
height: 50%;
}
+.absolutelyPositioned {
+ position: absolute;
+}
+
.indefiniteSize {
width: -webkit-fit-content;
height: auto;
</div>
</div>
+ <div class="indefiniteSize">
+ <div class="grid absolutelyPositioned">
+ <div class="firstRowFirstColumn sizedToGridArea" data-expected-width="0" data-expected-height="0">XX</div>
+ <div class="firstRowSecondColumn sizedToGridArea" data-expected-width="0" data-expected-height="0">XXXXX</div>
+ <div class="firstRowThirdColumn sizedToGridArea" data-expected-width="0" data-expected-height="0">XXX</div>
+ </div>
+ </div>
+
+ <div class="unconstrainedContainer">
+ <div class="grid absolutelyPositioned">
+ <div class="firstRowFirstColumn sizedToGridArea" data-expected-width="0" data-expected-height="0">XX</div>
+ <div class="firstRowSecondColumn sizedToGridArea" data-expected-width="0" data-expected-height="0">XXXXX</div>
+ <div class="firstRowThirdColumn sizedToGridArea" data-expected-width="0" data-expected-height="0">XXX</div>
+ </div>
+ </div>
+
<div class="indefiniteSize">
<div class="grid indefiniteSize">
<div class="firstRowFirstColumn sizedToGridArea" data-expected-width="20" data-expected-height="10">XX</div>
+2015-04-26 Manuel Rego Casasnovas <rego@igalia.com>
+
+ [CSS Grid Layout] LayoutBox::hasDefiniteLogicalHeight() should consider abspos boxes as definite
+ https://bugs.webkit.org/show_bug.cgi?id=144144
+
+ Reviewed by Darin Adler.
+
+ Change order in hasDefiniteLogicalHeight() ifs to consider definite
+ absolutely positioned elements.
+
+ Test: fast/css-grid-layout/percent-track-breadths-regarding-container-size.html
+
+ * rendering/RenderBox.cpp:
+ (WebCore::RenderBox::hasDefiniteLogicalHeight):
+
2015-04-26 Said Abou-Hallawa <sabouhallawa@apple.com>
SVGFilterBuilder should drive the builtin sourceAlpha from the passed sourceGraphic
bool RenderBox::hasDefiniteLogicalHeight() const
{
const Length& logicalHeight = style().logicalHeight();
- if (logicalHeight.isIntrinsicOrAuto())
- return false;
if (logicalHeight.isFixed())
return true;
// The size of the containing block of an absolutely positioned element is always definite with respect to that
if (hasOverrideContainingBlockLogicalHeight())
return overrideContainingBlockContentLogicalHeight() != -1;
#endif
+ if (logicalHeight.isIntrinsicOrAuto())
+ return false;
return percentageLogicalHeightIsResolvable(this);
}