[webkit-reviews] review denied: [Bug 69808] Implement -webkit-flex-align for cross axis alignment : [Attachment 110575] Patch

[bugzilla-daemon at webkit.org]

Dave Hyatt <hyatt at apple.com> has denied Tony Chang <tony at chromium.org>'s
request for review:
Bug 69808: Implement -webkit-flex-align for cross axis alignment
https://bugs.webkit.org/show_bug.cgi?id=69808

Attachment 110575: Patch
https://bugs.webkit.org/attachment.cgi?id=110575&action=review

------- Additional Comments from Dave Hyatt <hyatt at apple.com>
View in context: https://bugs.webkit.org/attachment.cgi?id=110575&action=review


> Source/WebCore/rendering/RenderFlexibleBox.cpp:571
> +	   case AlignStretch: {
> +	       Length height = isHorizontalFlow() ? child->style()->height() :
child->style()->width();
> +	       if (height.isAuto())
> +		   child->setLogicalHeight(child->logicalHeight() +
RenderFlexibleBox::availableLogicalHeightForChild(child));
> +	       break;

This is wrong if the child is a perpendicular writing mode. You need
setLogicalHeightForChild and to use logicalHeightForChild, or you could just
use physical setters.