[webkit-reviews] review requested: [Bug 87905] Can avoid a second layout of flex-items in some cases : [Attachment 174091] Patch

[bugzilla-daemon at webkit.org]

Carlos Garcia Campos <cgarcia at igalia.com> has asked  for review:
Bug 87905: Can avoid a second layout of flex-items in some cases
https://bugs.webkit.org/show_bug.cgi?id=87905

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

------- Additional Comments from Carlos Garcia Campos <cgarcia at igalia.com>
I'm getting a failure in 3 tests that I'm not sure why.

css3/flexbox/inline-flex-crash.html
css3/flexbox/inline-flex-crash2.html

The output is different in both cases, but it doesn't crash.

css3/flexbox/auto-margins.html

The output is correct, but the first time it's called it fails to get the
margin values, although the inspector shows the right ones and simply reloading
the page fixes the test. 

All other css3 flexbox tests pass.