Bug 82128 – Implement new flex property and deprecate flex function

• Status: RESOLVED
• Resolution: FIXED
• Component: CSS

The new flexbox spec was released on 22nd of March.

The flex function has been removed in favour of the flex property
http://www.w3.org/TR/css3-flexbox/#flexibility

[Tony Chang]

Created attachment 135412 [details]
Patch

[Tony Chang]

The code change was surprisingly small.  I left the old css parsing in this change.  I'll remove it separately.

[Ojan Vafai]

Comment on attachment 135412 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=135412&action=review

> Source/WebCore/rendering/RenderFlexibleBox.cpp:348
> +Length RenderFlexibleBox::preferredFlexLengthForChild(RenderBox* child) const

Not a huge fan of this name. I feel like it could easily be confused with the length after we've done the flex algorithm. How about mainAxisPreferredLengthForChild?

> Source/WebCore/rendering/RenderFlexibleBox.cpp:650
>  float RenderFlexibleBox::positiveFlexForChild(RenderBox* child) const
>  {
> -    return isHorizontalFlow() ? child->style()->flexboxWidthPositiveFlex() : child->style()->flexboxHeightPositiveFlex();
> +    return child->style()->positiveFlex();
>  }
>  
>  float RenderFlexibleBox::negativeFlexForChild(RenderBox* child) const
>  {
> -    return isHorizontalFlow() ? child->style()->flexboxWidthNegativeFlex() : child->style()->flexboxHeightNegativeFlex();
> +    return child->style()->negativeFlex();
>  }

Don't necessarily need to do it in this patch, but these helper functions don't seem terribly useful to me.

[Tony Chang]

Created attachment 135419 [details]
Patch for landing

[Tony Chang]

(In reply to comment #3)
> (From update of attachment 135412 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=135412&action=review
> 
> > Source/WebCore/rendering/RenderFlexibleBox.cpp:348
> > +Length RenderFlexibleBox::preferredFlexLengthForChild(RenderBox* child) const
> 
> Not a huge fan of this name. I feel like it could easily be confused with the length after we've done the flex algorithm. How about mainAxisPreferredLengthForChild?

Renamed to preferredLengthForChild.

> > Source/WebCore/rendering/RenderFlexibleBox.cpp:650
> >  float RenderFlexibleBox::positiveFlexForChild(RenderBox* child) const
> >  {
> > -    return isHorizontalFlow() ? child->style()->flexboxWidthPositiveFlex() : child->style()->flexboxHeightPositiveFlex();
> > +    return child->style()->positiveFlex();
> >  }
> >  
> >  float RenderFlexibleBox::negativeFlexForChild(RenderBox* child) const
> >  {
> > -    return isHorizontalFlow() ? child->style()->flexboxWidthNegativeFlex() : child->style()->flexboxHeightNegativeFlex();
> > +    return child->style()->negativeFlex();
> >  }
> 
> Don't necessarily need to do it in this patch, but these helper functions don't seem terribly useful to me.

Inlined.

[WebKit Review Bot]

Comment on attachment 135419 [details]
Patch for landing

Clearing flags on attachment: 135419

Committed r113097: <http://trac.webkit.org/changeset/113097>

[WebKit Review Bot]

All reviewed patches have been landed.  Closing bug.