CSS Shapes Level 1 (CR) only contains shape-outside. We are focusing our efforts on finalizing the implementation of shape-outside, it's worth to remove shape-inside code at this point for now. A list of reasons for the removal: - Shape-inside is only part of Shapes Level 2, which needs to be improved on some topics. - Shape-inside is lack of new shapes support (e.g. inset). - Deprecated shapes (r165472) are removed from the code (e.g. rectangle), which affects shape-inside. - The current shape-inside code spreads across the layout code. - The current shape-inside implementation is experimental in some areas, and the partially implemented code can have security implications. - Removal of shape-inside opens possibilities for code complexity and performance optimizations for shape-outside. (e.g. simpler geometry code) If there is no objections, I'll remove the code for now, so we can start working on the mentioned optimizations. I'll also close the bugs in bugzilla which are affected by the change.
Created attachment 227695 [details] Patch
Created attachment 227702 [details] Patch
Created attachment 227705 [details] Patch
Once it is reviewed, I'll send out a mail to webkit-dev and wait 24hrs before landing.
Comment on attachment 227705 [details] Patch r=me
Created attachment 227850 [details] Patch
Created attachment 227863 [details] Patch for landing
Comment on attachment 227863 [details] Patch for landing Clearing flags on attachment: 227863 Committed r166301: <http://trac.webkit.org/changeset/166301>
All reviewed patches have been landed. Closing bug.