<div dir="ltr">Hi there,<div><br></div><div>The patch is landed in <a href="http://trac.webkit.org/changeset/166301" target="_blank">http://trac.webkit.org/changeset/166301</a></div><div>I already did the first step to simplify our geometry codebase in <a href="http://trac.webkit.org/changeset/166316" target="_blank">http://trac.webkit.org/changeset/166316</a></div>

<div><br></div><div>Thumbs up for our shiny shape-outside! :-)</div><div><br></div><div>Cheers,</div><div>Zoltan</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Mar 25, 2014 at 12:59 PM, Zoltan Horvath <span dir="ltr">&lt;<a href="mailto:zoltan@webkit.org" target="_blank">zoltan@webkit.org</a>&gt;</span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><pre style="width:auto"><div><font face="arial"><span style="white-space:normal">Hi there,</span></font></div>
<div><font face="arial"><span style="white-space:normal"><br></span></font></div>
<div><font face="arial"><span style="white-space:normal">CSS Shapes Level 1 (CR [1]) only contains shape-outside. We are focusing our effort on finalizing the implementation of shape-outside, so it&#39;s worth to remove shape-inside code at this point for now.</span></font></div>

<div><font face="arial"><span style="white-space:normal"><br></span></font></div><div><font face="arial"><span style="white-space:normal">A list of reasons for the removal:</span></font></div><div><font face="arial"><span style="white-space:normal"><br>

</span></font></div><div><font face="arial"><span style="white-space:normal">- Shape-inside is only part of Shapes Level 2, which needs to be improved on some topics.</span></font></div><div><font face="arial"><span style="white-space:normal">- Shape-inside is lack of new shapes support (e.g. inset).</span></font></div>

<div><font face="arial"><span style="white-space:normal">- Deprecated shapes (r165472) are removed from the code (e.g. rectangle), which affects shape-inside. </span></font></div><div><font face="arial"><span style="white-space:normal">- The current shape-inside code spreads across the layout code.</span></font></div>

<div><font face="arial"><span style="white-space:normal">- The current shape-inside implementation is experimental in some areas, and the partially implemented code can have security implications.</span></font></div><div>

<font face="arial"><span style="white-space:normal">- Removal of shape-inside opens possibilities for code complexity and performance optimizations for shape-outside. (e.g. simpler geometry code)</span></font></div><div>
<font face="arial"><span style="white-space:normal"><br>
</span></font></div><div><font face="arial"><span style="white-space:normal">I&#39;ve got the patch reviewed in bug #130698 [2] ready for landing. If there are no objections, I&#39;ll land the patch tomorrow morning.</span></font></div>

<div><font face="arial"><span style="white-space:normal"><br></span></font></div><div><font face="arial"><span style="white-space:normal">Cheers,</span></font></div><div><font face="arial"><span style="white-space:normal">Zoltan</span></font></div>

<div><font face="arial"><span style="white-space:normal"><br></span></font></div><div><font face="arial"><span style="white-space:normal">[1] <a href="http://www.w3.org/TR/2014/CR-css-shapes-1-20140320/" target="_blank">http://www.w3.org/TR/2014/CR-css-shapes-1-20140320/</a></span></font></div>

<div><font face="arial"><span style="white-space:normal">[2] <a href="https://bugs.webkit.org/show_bug.cgi?id=130698" target="_blank">https://bugs.webkit.org/show_bug.cgi?id=130698</a></span></font></div></pre></div></div>

</blockquote></div><br></div>