When we decide what should be the radius of a circle, we should choose the smallest available space. For instance when the width or height is not resolvable for the circle we should not have a radius for it.
Created attachment 194961 [details] proposed patch
*** Bug 111609 has been marked as a duplicate of this bug. ***
Comment on attachment 194961 [details] proposed patch View in context: https://bugs.webkit.org/attachment.cgi?id=194961&action=review > LayoutTests/fast/exclusions/shape-inside/shape-inside-on-nested-container-with-unresolved-height.html:6 > + font: 14px Ahem, sans-serif; Not really needed as it's a ref test and I would expect the font resolution to give similar results on both tests. It also makes the output unreadable by maintainers. (and btw using 14px as the font size makes little sense, it's a lot easier to use a round number like 10px or 15px so that it's easier to know where the line layout algorithm breaks). > LayoutTests/fast/exclusions/shape-inside/shape-inside-on-nested-container-with-unresolved-height.html:9 > + height: 300px; This is unneeded and should be removed. > LayoutTests/fast/exclusions/shape-inside/shape-inside-on-nested-container-with-unresolved-height.html:28 > +<p>Requires Ahem font. Since the nested container's height is not resolvable, we shouldn't apply the circle shape-inside on the content.</p> No need for Ahem. > LayoutTests/fast/exclusions/shape-inside/shape-inside-on-nested-container-with-unresolved-height.html:29 > +<p>Bug <a href="http://webkit.org/b/113255 ">113255</a>: The radius of a circle should be computed based on the shorter available dimensions</p> Please add a description (ie what is expected in the output) before landing.
Landed in http://trac.webkit.org/changeset/146938